[PATCH] D148115: [WebAssembly] Handle DebugLoc in DebugValueManager

[Heejin Ahn via Phabricator via llvm-commits]

aheejin created this revision.
aheejin added a reviewer: dschuff.
Herald added subscribers: pmatos, asb, wingo, ecnelises, sunfish, hiraditya, jgravelle-google, sbc100.
Herald added a project: All.
aheejin requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

According to
https://llvm.org/docs/HowToUpdateDebugInfo.html#when-to-preserve-an-instruction-location,
when moving (and in our case cloning) within the same BB, the debug
location is preserved. But when moving / cloning to a different BB, we
preserve the debug location only if the destination BB contains the same
location. Currently we preserve the debug loc unconditionally in all
cases. This CL correctly handles the debug locs in DebugValueManager.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D148115

Files:
  llvm/lib/Target/WebAssembly/WebAssemblyDebugValueManager.cpp
  llvm/test/DebugInfo/WebAssembly/dbg-loc-reg-stackify.mir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D148115.512776.patch
Type: text/x-patch
Size: 5969 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230412/de9baa60/attachment.bin>