mgorny added a reviewer: doug.gregor.
mgorny added a subscriber: doug.gregor.
mgorny added a comment.
If I read the git correctly, the change that forbid defining builtins was
initially made in https://reviews.llvm.org/rL64639. @doug.gregor, any chance
you could help us over here? Is clang
joerg added a comment.
Note that the normal compiler-rt functions have a different name than the
builtins they provide, at least from the C frontend view.
Repository:
rL LLVM
https://reviews.llvm.org/D30025
___
cfe-commits mailing list
theraven added a comment.
No, it's a bug in clang. Clang does not reject other functions that are used
to implement builtins (if it did, compiler-rt would be a lot more difficult to
build).
Repository:
rL LLVM
https://reviews.llvm.org/D30025
mgorny added a comment.
In https://reviews.llvm.org/D30025#678462, @theraven wrote:
> This code is working around something that's probably a clang bug. It would
> be better to fix the clang bug than add more complex workarounds.
Well, clang explicitly rejects those functions as errors, so I
mgorny created this revision.
Herald added a subscriber: dberris.
Make the use of #pragma redefine_extname and appropriate renames of
builtins conditional to using clang. GCC used not to support it outside
Solaris and currently seems to be very restrictive on applying it.
In other words, it does