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