[PATCH] D11933: Extending debug ranges

[Vikram TV via llvm-commits]

On Thu, Aug 20, 2015 at 5:58 AM, Alex Lorenz <arphaman at gmail.com> wrote:

> arphaman added inline comments.
>
> ================
> Comment at: test/DebugInfo/MIR/extend-debug-range.mir:249
> @@ +248,3 @@
> +
> +    MOV32mr %rip, 1, _, @m, _, %ecx, debug-location !37 :: (store 4 into
> %ir.m)
> +    dead undef %edi = MOV32ri64 @.str, implicit-def %rdi, debug-location
> !38
> ----------------
> 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:
>
>      MOV32mr %rip, 1, _, @m, _, %ecx, debug-location !37 :: (store 4 into
> @m)
>
>
> http://reviews.llvm.org/D11933
>
>
>
>
Hi,

I updated to latest llvm and made the change you suggested. But with
-march=x86, I see an error like this:
error: extend-debug-range.mir:191:33: invalid DWARF register
    CFI_INSTRUCTION .cfi_offset %rbp, -16

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.


-- 

Good time...
Vikram TV
CompilerTree Technologies
Mysore, Karnataka, INDIA
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150820/c81f9110/attachment.html>