This is an archive of the discontinued LLVM Phabricator instance.

[LLD][ELF][NewPM] Add option to force legacy PM
ClosedPublic

Authored by aeubanks on Dec 1 2020, 11:52 AM.

Diff Detail

Event Timeline

aeubanks created this revision.Dec 1 2020, 11:52 AM
aeubanks requested review of this revision.Dec 1 2020, 11:52 AM
aeubanks retitled this revision from [LTO][NewPM] Add option to force legacy PM to [LLD][ELF][NewPM] Add option to force legacy PM.Dec 1 2020, 12:00 PM
MaskRay added inline comments.Dec 1 2020, 12:41 PM
lld/ELF/Options.td
529

This should probably be --no-lto-new-pass-manager.

FF -> BB

aeubanks updated this revision to Diff 308745.Dec 1 2020, 12:50 PM

use BB

lld/ELF/Options.td
529

Thanks, I wasn't aware of that.

MaskRay accepted this revision.Dec 1 2020, 1:28 PM

LGTM.

lld/test/ELF/lto/new-pass-manager.ll
9

%t2.o -> /dev/null since the output is not used

This revision is now accepted and ready to land.Dec 1 2020, 1:28 PM
This revision was automatically updated to reflect the committed changes.