[PATCH] D46940: [ASTImporter] make sure that ACtx::getParents still works
Gabor Marton via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri May 18 07:47:18 PDT 2018
martong added inline comments.
================
Comment at: lib/AST/ASTImporter.cpp:6776
+ // been invalidated to avoid repeatedly calling this.
+ ToContext.invalidateParents();
+
----------------
Can an `Expr` has a parent too? If yes, why not invalidate the parents in `Import(Expr*)` ?
I am also a bit concerned to call the invalidation during the import of all `Stmt` (and possible during the import of `Expr`s too).
Isn't it enough to invalidate only during `Import(Decl*)`?
https://reviews.llvm.org/D46940
More information about the cfe-commits
mailing list