lebedev.ri added a comment.

In D67159#1659084 <https://reviews.llvm.org/D67159#1659084>, @simon_tatham 
wrote:

> On the general discomfort with this attribute existing: I'd be happy to lock 
> it down, or mark it as "not recommended" in some way, if that's any help. I 
> don't personally intend any use of it outside a single system header file 
> (namely `arm_mve.h`, which D67161 <https://reviews.llvm.org/D67161> will 
> introduce the initial version of).


I think that would at least be something - make it print big fat undisableable 
warning if used outside of a system header.

> A warning along the lines of "don't use this!", automatically suppressed by 
> the usual change of warning settings in system headers, would seem like a 
> perfectly reasonable precaution, for example.




Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67159/new/

https://reviews.llvm.org/D67159



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

Reply via email to