[llvm] [gold] Add a new `-plugin-opt` to enable `MCTargetOptions::AsmVerbose` (PR #71606)

Teresa Johnson via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 8 06:25:33 PST 2023


teresajohnson wrote:

> I see that lld enables this unconditionally:
> 
> https://github.com/llvm/llvm-project/blob/26ab444e88fc8fdd554e5a9381a68b7b5e63b6fd/lld/ELF/LTO.cpp#L156
> 
> How about just make it unconditionally true to be consistent and avoid adding another plugin option?

In which case you can just modify https://github.com/llvm/llvm-project/blob/26ab444e88fc8fdd554e5a9381a68b7b5e63b6fd/llvm/test/tools/gold/X86/emit-asm.ll slightly to check that the comments are emitted.

https://github.com/llvm/llvm-project/pull/71606


More information about the llvm-commits mailing list