[PATCH] D58814: [clang][Index] Constructors and Destructors do not reference class
Nathan Hawes via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Mar 1 09:56:51 PST 2019
nathawes requested changes to this revision.
nathawes added a comment.
This revision now requires changes to proceed.
These references were added to support using the index data for symbol rename. I.e. so that when you rename the class, you can use the index data to find all occurrences of its name, including its use in constructor/destructor declarations and references (hence the test cases in test/Index/Core/index-source.cpp). If you need to specifically find references of the class symbol, as opposed to its name, could we instead distinguish these cases with a more specific SymbolRole, e.g. NameReference as opposed to Reference, and filter them based on that instead?
CHANGES SINCE LAST ACTION
More information about the cfe-commits