[llvm] [AMDGPU][RFC] Combine asm and disasm tests. (PR #90214)
Fangrui Song via llvm-commits
llvm-commits at lists.llvm.org
Fri Jul 5 14:17:41 PDT 2024
MaskRay wrote:
#92895, which does not add another MCStreamer, looks fine to me.
Note, we really should not add another MCStreamer, which is very easy to be misused.
MCAsmStreamer has some existing issues related to expression evaluation. If another MCStreamer is added, I am concerned others might use it for more assembler tasks (e.g. to support `.section`), which make future maintenance more difficult.
If the llvm-objdump output isn't convenient for AMDGPU, I think it is fine to add a new output mode.
https://github.com/llvm/llvm-project/pull/90214
More information about the llvm-commits
mailing list