[PATCH] D113840: [Driver][Android] Remove unneeded isNoExecStackDefault

2021-11-17 Thread Fangrui Song via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG062ef8f6b472: [Driver][Android] Remove unneeded isNoExecStackDefault (authored by MaskRay). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D113840/new/

[PATCH] D113840: [Driver][Android] Remove unneeded isNoExecStackDefault

2021-11-17 Thread Dan Albert via Phabricator via cfe-commits
danalbert accepted this revision. danalbert added a comment. This revision is now accepted and ready to land. Fine by me if this is the default for LLD. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D113840/new/ https://reviews.llvm.org/D113840

[PATCH] D113840: [Driver][Android] Remove unneeded isNoExecStackDefault

2021-11-16 Thread Pirama Arumuga Nainar via Phabricator via cfe-commits
pirama edited reviewers, added: srhines; removed: pirama. pirama added a comment. Will defer to Dan's review. I think these changes are safe for the Android platform (where we only use lld). The NDK only supports lld but that does not preclude external users from using other linkers.

[PATCH] D113840: [Driver][Android] Remove unneeded isNoExecStackDefault

2021-11-13 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay created this revision. MaskRay added reviewers: danalbert, pirama. Herald added a subscriber: danielkiss. MaskRay requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. ld.lld used by Android ignores .note.GNU-stack and defaults to