[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