[WebAssembly] Print DEBUG_VALUE once for target indices
authorHeejin Ahn <aheejin@gmail.com>
Wed, 7 Dec 2022 18:51:06 +0000 (10:51 -0800)
committerHeejin Ahn <aheejin@gmail.com>
Fri, 9 Dec 2022 18:57:32 +0000 (10:57 -0800)
commitae74e59f701724712869e9c50de124c2fe6790c8
treec97e862a2397e7e16dc5693aab3475b17de12558
parentf23f26032d136cce25a3711afb92203dbe6fdeba
[WebAssembly] Print DEBUG_VALUE once for target indices

`DEBUG_VALUE` comments are printed before an instruction, so they are
not printed with `AddComment` method as other comments are, but printed
using `emitRawComment` method. But currently `emitDebugValueComment`
calls `emitRawComment` twice for target-index-based `DBG_VALUE`s: once
in the `switch`-`case`,
https://github.com/llvm/llvm-project/blob/d77ae7f2513504655e555cd326208598093d66e2/llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp#L1192-L1193
and again at the end of the method:
https://github.com/llvm/llvm-project/blob/d77ae7f2513504655e555cd326208598093d66e2/llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp#L1227-L1228

This makes them printed twice. I think this happened through multiple
commits modifying and refactoring this method.

Reviewed By: dschuff

Differential Revision: https://reviews.llvm.org/D139579
llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp