This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rGff11f0aa5de1: [Clang] Pass -z rel to linker for Fuchsia
(authored by mcgrathr).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
h
mcgrathr added a comment.
Reordered the switches.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D113136/new/
https://reviews.llvm.org/D113136
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https:
mcgrathr updated this revision to Diff 386293.
mcgrathr marked an inline comment as done.
mcgrathr added a comment.
reordered switches
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D113136/new/
https://reviews.llvm.org/D113136
Files:
clang/lib/Dr
phosek accepted this revision.
phosek added a comment.
This revision is now accepted and ready to land.
LGTM
Comment at: clang/lib/Driver/ToolChains/Fuchsia.cpp:64-65
CmdArgs.push_back("--pack-dyn-relocs=relr");
+CmdArgs.push_back("-z");
+CmdArgs.push_back("rel");
mcgrathr updated this revision to Diff 386262.
mcgrathr added a comment.
rebased
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D113136/new/
https://reviews.llvm.org/D113136
Files:
clang/lib/Driver/ToolChains/Fuchsia.cpp
clang/test/Driver/fuchsi
mcgrathr created this revision.
mcgrathr added a reviewer: phosek.
Herald added a subscriber: abrachet.
mcgrathr requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
Fuchsia already supports the more compact relocation format.
Make it the defaul
