<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Aug 21, 2015 at 2:30 AM, Alex L <span dir="ltr"><<a href="mailto:arphaman@gmail.com" target="_blank">arphaman@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span>2015-08-20 9:42 GMT-07:00 Vikram TV <span dir="ltr"><<a href="mailto:vikram.tarikere@gmail.com" target="_blank">vikram.tarikere@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div><div><br><div class="gmail_quote">On Thu, Aug 20, 2015 at 5:58 AM, Alex Lorenz <span dir="ltr"><<a href="mailto:arphaman@gmail.com" target="_blank">arphaman@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">arphaman added inline comments.<br>
<br>
================<br>
Comment at: test/DebugInfo/MIR/extend-debug-range.mir:249<br>
@@ +248,3 @@<br>
+<br>
+ MOV32mr %rip, 1, _, @m, _, %ecx, debug-location !37 :: (store 4 into %ir.m)<br>
+ dead undef %edi = MOV32ri64 @.str, implicit-def %rdi, debug-location !38<br>
----------------<br>
I tweaked the syntax for the memory operands slightly while working on the fix for the problem you reported, so this instruction should be as follows now:<br>
<br>
MOV32mr %rip, 1, _, @m, _, %ecx, debug-location !37 :: (store 4 into @m)<br>
<br>
<br>
<a href="http://reviews.llvm.org/D11933" rel="noreferrer" target="_blank">http://reviews.llvm.org/D11933</a><br>
<br>
<br>
<br>
</blockquote></div><div class="gmail_extra"><br></div></div></div>Hi,</div><div class="gmail_extra"><br></div><div class="gmail_extra">I updated to latest llvm and made the change you suggested. But with -march=x86, I see an error like this:</div><div class="gmail_extra"><div class="gmail_extra">error: extend-debug-range.mir:191:33: invalid DWARF register</div><div class="gmail_extra"> CFI_INSTRUCTION .cfi_offset %rbp, -16</div></div></div></blockquote><div><br></div></span><div>The file was produced for the x86-64 architecture, so you have to use -march=x86-64 in order for run llc on it. The target directory should still be 'X86' though.</div><span><div><br></div></span></div></div></div></blockquote><div>Okay! I should have tried x86-64. Sorry about that. </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span><div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div><br></div><div>Is it okay to consider the current patch without .mir test cases? I will send a separate patch with .mir test cases once MIR serialization is stable.</div></div></div></blockquote><div><br></div></span><div>I would be ok with that.</div></div></div></div></blockquote><div> </div><div>I will send an updated patch. <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><br></div><div>Cheers,</div><div>Alex</div><span><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><span><br clear="all"><div><br></div>-- <br><div><div dir="ltr"><div><div dir="ltr"><div><br></div><div>Good time...</div><div>Vikram TV</div><div>CompilerTree Technologies</div><div>Mysore, Karnataka, INDIA</div></div></div></div></div>
</span></div></div>
</blockquote></span></div><br></div></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div><div dir="ltr"><div><div dir="ltr"><div><br></div><div>Good time...</div><div>Vikram TV</div><div>CompilerTree Technologies</div><div>Mysore, Karnataka, INDIA</div></div></div></div></div>
</div></div>