NoQ added a comment.

Yay.

Thanks Gabor, i didn't think about it but it look very nice to have such 
matcher.

I think the matcher shouldn't be checker-local, but shared into `ASTMatchers.h` 
all other matchers - not only this matcher is universally useful, but also i 
have weird vague memories that matchers may collide by name if defined in 
different translation units differently (if this is indeed this way then having 
the same matcher in multiple places is very bad).


https://reviews.llvm.org/D38921



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

Reply via email to