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

Paul Kirth via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 24 14:05:33 PDT 2023


paulkirth added a comment.

@peter.smith @nikic @maskray @tejohnson, given that Rust and others seem to use the `.llvmbc`, I see a few ways we can move forward here.

1. have a hard transition (e..g., this patch), and make downstream consumers adapt now.
2. have a soft transition, and mark `.llvmbc` as deprecated, then remove in the following release (so about 6 months from now).
3. forgo trying to separate the old uses of `.llvmbc` from new FatLTO uses and just reuse the `.llvmbc` section name.
4. ... something completely different?

I'm not really sure what the best way to go about this would be. I'd lean towards a soft transition (2), since that seems more forgiving and less hostile towards downstream consumers, but I think there are merits for any of the approaches.  There are probably other options I haven't considered either, so thoughts and suggestions would be quite welcome.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D152973



More information about the llvm-commits mailing list