[PATCH] D154923: [CodeGen] Support bitcode input containing multiple modules
Fangrui Song via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jul 11 16:11:15 PDT 2023
MaskRay added a comment.
In D154923#4491411 <https://reviews.llvm.org/D154923#4491411>, @efriedma wrote:
> If I follow correctly, this is basically undoing the splitting that was done by the command that produced the bitcode file?
Yes, undoing `llvm/lib/Transforms/IPO/ThinLTOBitcodeWriter.cpp` change that would make the output bitcode file not usable as an input for non-LTO use cases.
> I guess that could be useful. But it requires either renaming your object files from the default ".o" to ".bc", or explicitly passing "-x ir"? That seems unintuitive. Maybe it's better to put this behind some explicit flag?
Yes, specify `-x ir` or let the driver deduce the file type with the predefined extension `.bc`.
I think this is a less common operation (compiling with LTO but then using as non-LTO), so I think adding another option seems not necessary.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D154923/new/
https://reviews.llvm.org/D154923
More information about the cfe-commits
mailing list