[PATCH] D38140: [DebugInfo] Do not extend range for physreg in LiveDebugVariables

[Bjorn Pettersson via Phabricator via llvm-commits]

bjope added inline comments.


================
Comment at: test/DebugInfo/MIR/X86/live-debug-vars-unused-arg.mir:1
+# RUN: llc -O1 -start-before=greedy -stop-before=livedebugvalues -o /dev/null %s -debug-only=livedebugvars 2>&1 | FileCheck %s
+
----------------
aprantl wrote:
> If you really need to check the debug output of the pass (could we check the IR instead?) this needs a `REQUIRES: asserts`.
Well, this patch does not impact the IR/MIR by itself. So in order to have a test case that fails without the patch LiveDebugVariables I need to look at the debug printout (I do not think that it is possible to get the VRM and LDV data structures dumped in the MIR yet?).

I can however prepare a test case that would fail if we land https://reviews.llvm.org/D38229 without landing this patch first.


https://reviews.llvm.org/D38140