[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