[PATCH] D21626: Lit C++11 Compatibility Patch #10

This revision was automatically updated to reflect the committed changes. Closed by commit rL296193: [Test] Make Lit tests C++11 compatible #10 (authored by lcharles). Changed prior to commit: https://reviews.llvm.org/D21626?vs=89736&id=89737#toc Repository: rL LLVM https://reviews.llvm.org

[PATCH] D21626: Lit C++11 Compatibility Patch #10

tigerleapgorge marked 2 inline comments as done. tigerleapgorge added inline comments. Comment at: test/SemaCXX/PR9572.cpp:34 +// expected-error@-2 {{non-deleted function '~Bar' cannot override a deleted function}} +// expected-note@-3 {{while declaring the implicit destructor f

[PATCH] D21626: Lit C++11 Compatibility Patch #10

tigerleapgorge updated this revision to Diff 89736. tigerleapgorge added a comment. Updated patch in accordance to Richard Smith's comments. https://reviews.llvm.org/D21626 Files: test/Modules/Inputs/merge-using-decls/b.h test/Modules/merge-using-decls.cpp test/SemaCXX/PR9572.cpp test/S

[PATCH] D21626: Lit C++11 Compatibility Patch #10

rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land. LGTM with a couple of changes. Comment at: test/Modules/Inputs/merge-using-decls/a.h:25 +#if __cplusplus <= 199711L // C++11 does not allow access declerations template st

[PATCH] D21626: Lit C++11 Compatibility Patch #10

tigerleapgorge updated this revision to Diff 88232. tigerleapgorge edited the summary of this revision. tigerleapgorge added a comment. Remove 2 tests reviewed in https://reviews.llvm.org/D29685 https://reviews.llvm.org/D21626 Files: test/Modules/Inputs/merge-using-decls/a.h test/Modules/In

[PATCH] D21626: Lit C++11 Compatibility Patch #10

tigerleapgorge updated this revision to Diff 87711. tigerleapgorge edited the summary of this revision. tigerleapgorge added a comment. Remove 2 OpenMP tests reviewed via https://reviews.llvm.org/D29725 https://reviews.llvm.org/D21626 Files: test/Modules/Inputs/merge-using-decls/a.h test/Mo

[PATCH] D21626: Lit C++11 Compatibility Patch #10

tigerleapgorge updated this revision to Diff 87312. tigerleapgorge added a comment. Remove 2 tests reviewed under https://reviews.llvm.org/D29520. https://reviews.llvm.org/D21626 Files: test/Modules/Inputs/merge-using-decls/a.h test/Modules/Inputs/merge-using-decls/b.h test/Modules/merge-

[PATCH] D21626: Lit C++11 Compatibility Patch #10

tigerleapgorge updated this revision to Diff 81817. tigerleapgorge added a comment. Previous update contain typos in the RUN lines of test/SemaCXX/virtual-base-used.cpp This has been corrected in this latest revision. https://reviews.llvm.org/D21626 Files: test/Modules/Inputs/merge-using-dec

[PATCH] D21626: Lit C++11 Compatibility Patch #10

tigerleapgorge updated this revision to Diff 81810. tigerleapgorge added a comment. Update patch to match latest Clang C++11 diagnostics. 2 tests are changed. test/SemaCXX/PR9572.cpp Expect the following additional Note following existing Error. destructor of 'Foo' is implicitly deleted b

[PATCH] D21626: Lit C++11 Compatibility Patch #10

tigerleapgorge created this revision. tigerleapgorge added a reviewer: rsmith. tigerleapgorge added a subscriber: cfe-commits. Hi Everyone, I am continuing with updating the Lit tests for C++11 compatibility. 11 tests this time. test/Modules/Inputs/merge-using-decls/a.h test/Modules/Inputs/merg