This is an archive of the discontinued LLVM Phabricator instance.

[Wasm][LTO][NPM] Use NPM for LTO with ENABLE_EXPERIMENTAL_NEW_PASS_MANAGER
ClosedPublic

Authored by aeubanks on Dec 8 2020, 10:25 AM.

Diff Detail

Unit TestsFailed

Event Timeline

aeubanks created this revision.Dec 8 2020, 10:25 AM
aeubanks requested review of this revision.Dec 8 2020, 10:25 AM
Herald added a project: Restricted Project. · View Herald TranscriptDec 8 2020, 10:25 AM

On the ELF side we have switched to --no-lto-legacy-pass-manager --lto-legacy-pass-manager and dropped --[no-]lto-new-pass-manager...

The driver uses -plugin-opt={new,legacy}-pass-manager

I can do that in a follow up change, I'd like to land this separately since it's two different functionality changes.

MaskRay accepted this revision.Dec 14 2020, 9:34 AM

LGTM.

This revision is now accepted and ready to land.Dec 14 2020, 9:34 AM
This revision was landed with ongoing or failed builds.Dec 14 2020, 10:15 AM
This revision was automatically updated to reflect the committed changes.