[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