[PATCH] D62385: [WIP][DebugInfo] LiveDebugValues should allow different variable fragments to have different locations

[Jeremy Morse via Phabricator via llvm-commits]

jmorse created this revision.
Herald added subscribers: llvm-commits, javed.absar.
Herald added a project: LLVM.

[Work in progress, not really up for review]

Here's a patch that makes LiveDebugValues recognise different variable fragments as having different locations, allowing it to propagate fragments independently (see: the regression test).

There's an obvious flaw with this : *overlapping* fragments will be recognised as being different fragments, and will get propagated, leading to unsound locations being extended. What's not clear to me is whether this is best dealt with at a different layer, which I'll write up in the bug report at https://bugs.llvm.org/show_bug.cgi?id=41979


Repository:
  rL LLVM

https://reviews.llvm.org/D62385

Files:
  include/llvm/IR/DebugInfoMetadata.h
  lib/CodeGen/LiveDebugValues.cpp
  test/DebugInfo/ARM/partial-subreg.ll
  test/DebugInfo/MIR/X86/live-debug-values-fragments.mir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D62385.201215.patch
Type: text/x-patch
Size: 11381 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190524/1194573f/attachment.bin>