[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?

  rC Clang



More information about the cfe-commits mailing list