This is an archive of the discontinued LLVM Phabricator instance.

[Clang] Pass -z rel to linker for Fuchsia
ClosedPublic

Authored by mcgrathr on Nov 3 2021, 12:10 PM.

Details

Summary

Fuchsia already supports the more compact relocation format.
Make it the default.

Diff Detail

Event Timeline

mcgrathr created this revision.Nov 3 2021, 12:10 PM
mcgrathr requested review of this revision.Nov 3 2021, 12:10 PM
Herald added a project: Restricted Project. · View Herald TranscriptNov 3 2021, 12:10 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
phosek accepted this revision.Nov 10 2021, 11:29 AM

LGTM

clang/lib/Driver/ToolChains/Fuchsia.cpp
66–67

Super minor nit, but I'd prefer moving this two lines up to keep all the -z options together.

This revision is now accepted and ready to land.Nov 10 2021, 11:29 AM
mcgrathr updated this revision to Diff 386293.Nov 10 2021, 1:02 PM
mcgrathr marked an inline comment as done.

reordered switches

Reordered the switches.

This revision was landed with ongoing or failed builds.Nov 10 2021, 1:31 PM
This revision was automatically updated to reflect the committed changes.