danalbert added a comment.
The NDK still supports linkers other than LLD, but we are changing the default
to LLD in the next release. I'd prefer to keep this for the time being, but
don't feel strongly about it.
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D56456
thakis added a comment.
Do we still need this? I was surprised by clang not using faddrsing on Andriod
by default.
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D56456/new/
https://reviews.llvm.org/D56456
___
cfe-comm
This revision was automatically updated to reflect the committed changes.
Closed by commit rL350668: [Driver] Default to -fno-addrsig on Android.
(authored by danalbert, committed by ).
Herald added a subscriber: llvm-commits.
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llv
danalbert created this revision.
danalbert added reviewers: srhines, pirama.
The Android NDK still uses GNU binutils by default.
Repository:
rC Clang
https://reviews.llvm.org/D56456
Files:
lib/Driver/ToolChains/Clang.cpp
test/Driver/addrsig.c
Index: test/Driver/addrsig.c
==