Page MenuHomePhabricator

[Driver] Default to -fno-addrsig on Android.
ClosedPublic

Authored by danalbert on Jan 8 2019, 2:24 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

danalbert created this revision.Jan 8 2019, 2:24 PM
srhines accepted this revision.Jan 8 2019, 2:28 PM
This revision is now accepted and ready to land.Jan 8 2019, 2:28 PM
This revision was automatically updated to reflect the committed changes.

Do we still need this? I was surprised by clang not using faddrsing on Andriod by default.

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.