[PATCH] D27132: Do not do raw name replacement when FromDecl is a class forward-declaration.

Benjamin Kramer via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 25 08:08:55 PST 2016


bkramer accepted this revision.
bkramer added inline comments.
This revision is now accepted and ready to land.


================
Comment at: lib/Tooling/Core/Lookup.cpp:131
+  // still considered as referring to the original definition given the nature
+  // of forward-declarations, so we can't do a raw name replacement in this
+  // case.
----------------
the "given the nature" part doesn't add any useful information. Either drop it or explain in more detail what's the problem with forward decls.


https://reviews.llvm.org/D27132





More information about the cfe-commits mailing list