[PATCH] D53708: [ASTImporter] Add importer specific lookup
Gabor Marton via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Nov 30 05:47:36 PST 2018
martong added a comment.
Hey Alexei,
Thank you again for your comments. I changed to code accordingly.
Also, I recognized that we should use a `SmallSetVector` instead of a `SmallPtrSet`, because the latter results unpredictable iteration (pointers may have different values with each run). Though `SmallPtrSet` is more powerful than `SmallSetVector` it makes it really hard to debug false positive ODR warnings from the ASTImporter, because the warning may appear in one run but may not appear in the subsequent run. We need an iteration order in the order of insertions and that is produced by `SmallSetVector`.
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D53708/new/
https://reviews.llvm.org/D53708
More information about the cfe-commits
mailing list