[PATCH] D70576: [Clang] Always set -z now linker option on Fuchsia

2020-01-13 Thread Petr Hosek via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG231875e111fa: [Clang] Always set -z now linker option on Fuchsia (authored by phosek). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70576/new/ https://revi

[PATCH] D70576: [Clang] Always set -z now linker option on Fuchsia

2019-11-21 Thread Petr Hosek via Phabricator via cfe-commits
phosek added a comment. In D70576#1756049 , @leonardchan wrote: > LGTM. But just to confirm, did you test this on CQ/bots by adding `["now", > "-z"]` as linker flags to the `config("compiler")` in Fuchsia + Zircon? https://fuchsia-review.googlesource.co

[PATCH] D70576: [Clang] Always set -z now linker option on Fuchsia

2019-11-21 Thread Leonard Chan via Phabricator via cfe-commits
leonardchan accepted this revision. leonardchan added a comment. This revision is now accepted and ready to land. LGTM. But just to confirm, did you test this on CQ/bots by adding `["now", "-z"]` as linker flags to the `config("compiler")` in Fuchsia + Zircon? Repository: rG LLVM Github Monor

[PATCH] D70576: [Clang] Always set -z now linker option on Fuchsia

2019-11-21 Thread Petr Hosek via Phabricator via cfe-commits
phosek created this revision. phosek added reviewers: leonardchan, mcgrathr. Herald added a project: clang. Herald added a subscriber: cfe-commits. This should be the default on Fuchsia. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D70576 Files: clang/lib/Driver/ToolChains/