[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/ https:

[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. Reposit

[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 noexecs