[PATCH] D39722: [ASTImporter] Support TypeTraitExpr Importing

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 21 11:43:38 PST 2017


aaron.ballman added inline comments.


================
Comment at: lib/AST/ASTImporter.cpp:5622
+  SmallVector<TypeSourceInfo *, 4> ToArgVec;
+  for (auto FromArg : E->getArgs()) {
+    TypeSourceInfo *ToTI = Importer.Import(FromArg);
----------------
`const auto *`?


================
Comment at: lib/AST/ASTImporter.cpp:5631
+  // Value is always false.
+  bool ToValue = (!E->isValueDependent()) ? E->getValue() : false;
+
----------------
Remove spurious parens.


================
Comment at: unittests/AST/ASTImporterTest.cpp:548
+                                 typeTraitExpr(hasType(asString("int")))))))));
+}
 
----------------
Please add a value-dependent test as well.


https://reviews.llvm.org/D39722





More information about the cfe-commits mailing list