[PATCH] D42077: Ensure code complete with !LoadExternal sees all local decls.
Sam McCall via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Jan 15 09:33:42 PST 2018
sammccall created this revision.
sammccall added a reviewer: ilya-biryukov.
Herald added a subscriber: cfe-commits.
noload_lookups() was too lazy: in addition to avoiding external decls, it
avoided populating the lazy lookup structure for internal decls.
This is the right behavior for the existing callsite in ASTDumper, but I think
it's not a very useful default, so we populate it by default.
- remove an unused test file accidentally added in r322371.
- remove lookups_begin()/lookups_end() in favor of lookups().begin(), which is more common and more efficient.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 5717 bytes
Desc: not available
More information about the cfe-commits