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

Raphael Isemann via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 21 10:15:07 PDT 2018


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 for the input files.
Well all the 'Import/<node-name>' tests just try importing the respective AST node type and run checks on the node after importing. But we probably should reorganize the `test/Import` folder in the future, so then we can better document that the tests in the respective 'node-types` folder or so are supposed to do just this kind of check.


Repository:
  rC Clang

https://reviews.llvm.org/D50737





More information about the cfe-commits mailing list