[lld] [lld] Add `--lto-emit-asm` and alias `--plugin-opt=emit-llvm` to it (PR #97469)
Joseph Huber via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 2 13:15:35 PDT 2024
jhuber6 wrote:
> Thanks for catching this missing alias! I occasionally debug issues with `--lto-emit-asm`. I tried `--lto-emit-llvm`, but I didn't bother adding an alias then.
>
> Can you add a test to lto/emit-llvm.ll ?
>
> > Make --lto-emit-llvm to be more consistent with --lto-emit-asm
>
> I think "add --lto-emit-llvm and make --plugin-opt=emit-llvm" might be a better title. The body can mention that this improves consistency with --lto-emit-asm.
Test added, thanks for the quick review. I also wonder if we shouldn't make LLVM-IR a valid output type from the LTO interface so we don't need to use the hook for it. Maybe make it only valid under monolithic LTO.
https://github.com/llvm/llvm-project/pull/97469
More information about the llvm-commits
mailing list