[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