[PATCH] D152973: Reland "[gold] Add preliminary FatLTO support to the Gold plugin""

Mon Jul 24 14:52:16 PDT 2023

MaskRay added a comment.

In D152973#4529730 <https://reviews.llvm.org/D152973#4529730>, @paulkirth wrote:

> In D152973#4529666 <https://reviews.llvm.org/D152973#4529666>, @MaskRay wrote:
>
>> If `-ffat-lto-objects` doesn't need to go to the LLVM 17.0.0 release, we can do a soft transition:
>
> Given that Rust pins the LLVM revision to a stable release I think it may be good to get it into Clang 17. Doubly so, since it seems to me that they already have a version of FatLTO, and would benefit from using the FatLTO pipeline.

My suggestion is based on my (possibly wrong) understanding that rust-lang/rust only needs the LLVM side change, which is in `release/17.x`, but not Clang or lld changes :)
That said, if people feel that it's good to ensure clang and lld changes are in `release/17.x`, I'm totally fine as well.

(I'll go on a trip from Thursday onwards and won't be able to respond in time. Apologies if anyone needs me to do anything.)

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D152973