[PATCH] D41444: [ASTImporterTest] Make testing under '-fdelayed-template-parsing' mandatory

Aleksei Sidorin via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 21 05:09:18 PST 2017


a.sidorin added inline comments.


================
Comment at: unittests/AST/ASTImporterTest.cpp:32
+
+static RunOptions getRunOptionsForLanguage(Language Lang) {
+  ArgVector BasicArgs;
----------------
xazax.hun wrote:
> I wonder if in the future it would be worth to use something else, like LangOptions of clang instead of this enum. But I think this should be ok for now.
This was inspired by ASTMatchersTest. Reusing existing enum is nice but it is an item for a separate patch, I think.


Repository:
  rC Clang

https://reviews.llvm.org/D41444





More information about the cfe-commits mailing list