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

Adrian Prantl via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 25 09:08:18 PDT 2017


aprantl accepted this revision.
aprantl added a comment.
This revision is now accepted and ready to land.

This seems reasonable, thanks!
Could we make the test check the MIR instead of the debug output?



================
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
+
----------------
If you really need to check the debug output of the pass (could we check the IR instead?) this needs a `REQUIRES: asserts`.


https://reviews.llvm.org/D38140





More information about the llvm-commits mailing list