[PATCH] D67159: [clang] New __attribute__((__clang_builtin)).

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 5 03:50:45 PDT 2019


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





More information about the cfe-commits mailing list