[PATCH] D93110: [analyzer] Implement a first version of suppressions via attributes
    Valeriy Savchenko via Phabricator via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Fri Dec 11 05:57:40 PST 2020
    
    
  
vsavchenko added a comment.
Right now, I reused existing `suppress` attribute.  However I did it in a rather unconventional manner.  I allowed 0 arguments in one spelling and >1 in another, which seems odd.
I can see a couple other possible solutions here:
- Choose a "keyword" that would be used to suppress all warnings (e.g. "suppress(all)")
- Introduce two different suppress attributes (maybe even rename existing attribute to be GSLSuppress)
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D93110/new/
https://reviews.llvm.org/D93110
    
    
More information about the cfe-commits
mailing list