[PATCH] D38921: [analyzer] LoopUnrolling: update the matched assignment operators

Artem Dergachev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 30 04:43:43 PDT 2017


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





More information about the cfe-commits mailing list