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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to