[PATCH] D21666: [Tooling] Add optional argument to getFullyQualifiedName to prepend "::".

Sterling Augustine via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 24 11:04:40 PDT 2016


saugustine accepted this revision.
saugustine added a comment.
This revision is now accepted and ready to land.

This looks good and useful to me on the Tooling/Core side, but someone more familiar with the internals of NestedNamesSpecifiers should sign off on that portion.

Also, I would probably add a test case for _Bool, which is a case that has confused other logic like this. I think it will work, but may as well include the proof.


http://reviews.llvm.org/D21666





More information about the cfe-commits mailing list