[PATCH] D48027: [analyzer] Improve `CallDescription` to handle c++ method.

Henry Wong via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 22 06:29:03 PDT 2018


MTC added a comment.

In https://reviews.llvm.org/D48027#1207645, @xazax.hun wrote:

> Sorry for the delay, I think this is OK to commit.
>  As a possible improvement, I can imagine making it slightly stricter, e.g. you could only skip QualifiedNames for inline namespaces and the beginning. Maybe add support for staring with `""` or `::` to even disable skipping namespaces at the beginning?
>  But these are just nice to have features, I am perfectly ok with not having them or doing it in a followup patch.


Thanks, Gábor!
I will land it first and do the improvement according to the mismatch case in the followup patch!


https://reviews.llvm.org/D48027





More information about the cfe-commits mailing list