[PATCH] D58814: [clang][Index] Constructors and Destructors do not reference class

Kadir Cetinkaya via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 4 02:50:22 PST 2019


kadircet added a comment.

I agree with Dmitri's reasoning. I believe this should be handled inside the `renamer`s logic as mentioned in option 1. But if you believe this should not be the case, I am also happy to move logic of `adding a reference to class` on constructors&destructors to the consumer of renamer(as mentioned in option 2).


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58814/new/

https://reviews.llvm.org/D58814





More information about the cfe-commits mailing list