<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Mon, Aug 17, 2015 at 8:55 AM Adrian Prantl via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
> On Aug 17, 2015, at 7:05 AM, Vikram TV <<a href="mailto:vikram.tarikere@gmail.com" target="_blank">vikram.tarikere@gmail.com</a>> wrote:<br>
><br>
> tvvikram added inline comments.<br>
><br>
> ================<br>
> Comment at: lib/CodeGen/DebugValueFixup.cpp:2<br>
> @@ +1,3 @@<br>
> +//===-- DebugValueFixup.cpp - Fixup Debug Value MIs -----------------------===//<br>
> +//<br>
> +//                     The LLVM Compiler Infrastructure<br>
> ----------------<br>
> Is it okay to name the pass as "ExtendDebugValueRangeAndLocation" ?<br>
><br>
> ================<br>
> Comment at: test/DebugInfo/extend-debug-range.ll:1<br>
> @@ +1,2 @@<br>
> +; RUN: llc -print-machineinstrs=dbgval-fixup %s 2>&1 | FileCheck %s<br>
> +<br>
> ----------------<br>
> 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.<br>
><br>
<br>
+Alex, can you help Vikram in getting started with MIR serialization? I think this pass would be a perfect use case for it.<br><br></blockquote><div><br></div><div>I think we should test both for now, the MIR serialization isn't there yet and I'd prefer something that can be tested via looking at DWARF as well.</div><div><br></div><div>-eric </div></div></div>