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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to