MTC added a comment.

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

> Having C++ support would be awesome!
>  Thanks for working on this!
>
> While I do agree matching is not trivial with qualified names, this problem 
> is already solved with AST matchers.
>
> I wonder if using matchers or reusing the `HasNameMatcher` class would make 
> this easier. That code path is already well tested and optimized.


Thank you for pointing out the direction, xazax.hun!

I will looking into the matchers and try to give a better solution.


Repository:
  rC Clang

https://reviews.llvm.org/D48027



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to