logan added a comment.
Sorry, I don't have time to work on this recently. Here's the plan:
1. Add another flag to specify the path to `libclang_rt-builtin.a`.
2. If that flag was not specified and the compiler is `clang`, then run `clang
-###` to extract for `-L` option and `-l` option for
rengolin added a comment.
Was this abandoned? Merged?
https://reviews.llvm.org/D24083
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
rengolin added a comment.
In https://reviews.llvm.org/D24083#534464, @EricWF wrote:
> In https://reviews.llvm.org/D24083#534459, @logan wrote:
>
> > One solution might be adding the `libclang_rt.builtins.${arch}.a` detection
> > rules[1] to CMakeLists.txt, and manually specify
> >
EricWF added a comment.
In https://reviews.llvm.org/D24083#534459, @logan wrote:
> One solution might be adding the `libclang_rt.builtins.${arch}.a` detection
> rules[1] to CMakeLists.txt, and manually specify
> `-lclang_rt.builtins-${arch}.a` when `LIBCXXABI_USE_COMPILER_RT` is enabled.
>
logan added a comment.
Hi @rengolin and @EricWF,
After tracing the commit log of clang, it seems that `--rtlib=` is ignored
intentionally when `-nodefaultlibs` is present. (see also:
https://reviews.llvm.org/rL254535)
And, unfortunately, we don't have a command line option to ask clang to
EricWF added a comment.
This is not correct. We need the `-nodefaultlibs` on most platforms to prevent
libstdc++ from being linked. I think the correct fix is to change
`--rtlib=`
https://reviews.llvm.org/D24083
___
cfe-commits mailing list
rengolin added a comment.
This looks like a work around the fact that --rtlib is being ignored with
--nodefaultlibs. I'm not sure that's a sane behaviour, if you explicitly
specify --rtlib.
https://reviews.llvm.org/D24083
___
cfe-commits mailing
logan created this revision.
logan added reviewers: mclow.lists, rengolin, EricWF.
logan added a subscriber: cfe-commits.
Herald added a subscriber: aemerson.
This commit fixes __aeabi_idiv() link error when we are building
libc++abi with compiler-rt.
When compiler-rt is enabled, the option