Page MenuHomePhabricator

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

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

Details

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

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.

ormris updated this revision to Diff 424943.Apr 25 2022, 9:38 AM

Add context.