This is an archive of the discontinued LLVM Phabricator instance.

[LLD] [MinGW] Hook up more LTO options
ClosedPublic

Authored by mstorsjo on Aug 25 2023, 2:00 PM.

Details

Summary

Many of these options can be passed to the linker by the Clang
driver based on other options passed to Clang, after
a23bf1786be7c0738a4cf999c2957155bb32d5af. Before commit
5c92c9f34a7dba804479acef62c576d1a170ef1f, these were ignored by
lld, but now we're erroring out on the unrecognized options.

The ELF linker has even more LTO options available, but not
all of these are currently settable via the lld-link option
interface, and some aren't set automatically by Clang but only
if the user manually passes them - and thus probably aren't in
wide use so far. (Previously LLD/MinGW would have accepted them
silently but ignored them though.)

Diff Detail

Event Timeline

mstorsjo created this revision.Aug 25 2023, 2:00 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 25 2023, 2:00 PM
Herald added a subscriber: inglorion. · View Herald Transcript
mstorsjo requested review of this revision.Aug 25 2023, 2:00 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 25 2023, 2:01 PM
MaskRay accepted this revision.Aug 25 2023, 2:16 PM
This revision is now accepted and ready to land.Aug 25 2023, 2:16 PM

skimmed through. seems reasonable

This revision was automatically updated to reflect the committed changes.