[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