[PATCH] D123971: [clang] Enable SplitLTOUnits by default for SIE targets

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Apr 23 14:00:35 PDT 2022


MaskRay added a comment.

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.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D123971/new/

https://reviews.llvm.org/D123971



More information about the llvm-commits mailing list