Re: [PATCH] D21497: Fix test to specify C++03 (fails with C++11).

2016-06-20 Thread Paul Robinson via cfe-commits
probinson abandoned this revision. probinson added a comment. A proper fix is obviously better than patching the test! http://reviews.llvm.org/D21497 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listi

Re: [PATCH] D21497: Fix test to specify C++03 (fails with C++11).

2016-06-19 Thread Olivier Goffart via cfe-commits
ogoffart added a comment. I made http://reviews.llvm.org/D21502 that should also fix this test. http://reviews.llvm.org/D21497 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D21497: Fix test to specify C++03 (fails with C++11).

2016-06-19 Thread Olivier Goffart via cfe-commits
ogoffart added a comment. I'm sorry for commiting broken test. However this is supposed to work also in C++11 mode. I'm investigating what's wrong. http://reviews.llvm.org/D21497 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://list

[PATCH] D21497: Fix test to specify C++03 (fails with C++11).

2016-06-18 Thread Paul Robinson via cfe-commits
probinson created this revision. probinson added reviewers: rsmith, ogoffart. probinson added a subscriber: cfe-commits. Test fails if Clang defaults to C++11. I didn't track down exactly which RUN lines needed this, seemed simplest to put it on all of them. http://reviews.llvm.org/D21497 Fil