[PATCH] D11933: Extending debug ranges

[Vikram TV via llvm-commits]

tvvikram added inline comments.

================
Comment at: lib/CodeGen/DebugValueFixup.cpp:2
@@ +1,3 @@
+//===-- DebugValueFixup.cpp - Fixup Debug Value MIs -----------------------===//
+//
+//                     The LLVM Compiler Infrastructure
----------------
Is it okay to name the pass as "ExtendDebugValueRangeAndLocation" ?

================
Comment at: test/DebugInfo/extend-debug-range.ll:1
@@ +1,2 @@
+; RUN: llc -print-machineinstrs=dbgval-fixup %s 2>&1 | FileCheck %s
+
----------------
I am checking the generated machine instructions after the new pass to check for propagated debug ranges. But llc will also generate the .s file. Now, I tried using -stop-after option to llc, but it still generates the .s file.


http://reviews.llvm.org/D11933