[PATCH] D38694: [ASTImporter] Support importing CXXUnresolvedConstructExpr and UnresolvedLookupExpr

Aleksei Sidorin via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 4 07:49:20 PST 2017


a.sidorin added inline comments.


================
Comment at: lib/AST/ASTImporter.cpp:5877
+  DeclarationName Name = Importer.Import(E->getName());
+  if (E->getName().isEmpty() && Name.isEmpty())
+    return nullptr;
----------------
xazax.hun wrote:
> Is this condition correct?
Looks like it should be `E->getName() && !Name`.


https://reviews.llvm.org/D38694





More information about the cfe-commits mailing list