[clang] Enable SplitLTOUnits by default for SIE targets
Needs ReviewPublic
Actions

Authored by ormris on Apr 18 2022, 5:45 PM.

Details

Reviewers

tejohnson
pcc
mehdi_amini
probinson

Summary

This patch contains the SIE-specific parts of the unified LTO pipeline. See this conversation: https://discourse.llvm.org/t/rfc-a-unified-lto-bitcode-frontend/61774/15

Diff Detail

Event Timeline

ormris created this revision.Apr 18 2022, 5:45 PM

Herald added a project: Restricted Project. · View Herald TranscriptApr 18 2022, 5:45 PM

Herald added a subscriber: inglorion. · View Herald Transcript

ormris requested review of this revision.Apr 18 2022, 5:45 PM

Herald added a project: Restricted Project. · View Herald TranscriptApr 18 2022, 5:45 PM

Herald added subscribers: llvm-commits, MaskRay. · View Herald Transcript

Harbormaster completed remote builds in B160138: Diff 423498.Apr 18 2022, 5:45 PM

ormris added a parent revision: D123803: [WIP][llvm] A Unified LTO Bitcode Frontend.Apr 18 2022, 5:45 PM

ormris added a reviewer: probinson.Apr 19 2022, 4:19 PM

If a vendor wants to change some driver default, that's fine, especially when on/off already has a condition. But please include full context per https://llvm.org/docs/Phabricator.html#requesting-a-review-via-the-web-interface and add a clang/test/Driver/ test.
Also, "SIE" isn't a clear abbreviation. PS4 seems more understandable by other contributors.

Add context.

Harbormaster completed remote builds in B161191: Diff 424943.Apr 25 2022, 9:39 AM