[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