[PATCH] D50737: [ASTImporter] Add test for CXXNoexceptExpr

2018-08-21 Thread Raphael Isemann via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC340304: [ASTImporter] Add test for CXXNoexceptExpr (authored by teemperor, committed by ). Repository: rC Clang https://reviews.llvm.org/D50737 Files: test/Import/cxx-noexcept-expr/Inputs/F.cpp

[PATCH] D50737: [ASTImporter] Add test for CXXNoexceptExpr

2018-08-21 Thread Raphael Isemann via Phabricator via cfe-commits
teemperor added inline comments. Comment at: test/Import/cxx-noexcept-expr/test.cpp:1 +// RUN: clang-import-test -dump-ast -import %S/Inputs/F.cpp -expression %s | FileCheck %s + hiraditya wrote: > Can we add a line/comment to explain what this test does? Same

[PATCH] D50737: [ASTImporter] Add test for CXXNoexceptExpr

2018-08-19 Thread Aleksei Sidorin via Phabricator via cfe-commits
a_sidorin accepted this revision. a_sidorin added a comment. This revision is now accepted and ready to land. Thanks! Repository: rC Clang https://reviews.llvm.org/D50737 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D50737: [ASTImporter] Add test for CXXNoexceptExpr

2018-08-15 Thread Aditya Kumar via Phabricator via cfe-commits
hiraditya added inline comments. Comment at: test/Import/cxx-noexcept-expr/test.cpp:1 +// RUN: clang-import-test -dump-ast -import %S/Inputs/F.cpp -expression %s | FileCheck %s + Can we add a line/comment to explain what this test does? Same for the input

[PATCH] D50737: [ASTImporter] Add test for CXXNoexceptExpr

2018-08-14 Thread Raphael Isemann via Phabricator via cfe-commits
teemperor created this revision. Herald added a subscriber: martong. Herald added a reviewer: a.sidorin. Repository: rC Clang https://reviews.llvm.org/D50737 Files: test/Import/cxx-noexcept-expr/Inputs/F.cpp test/Import/cxx-noexcept-expr/test.cpp Index: