On Tue, Jan 30, 2018 at 11:53:40PM +0100, Jakub Jelinek wrote: > > DW_AT_data_location needs to be the data pointer, which is at offset 0, > but we were emitting a bogus +8 in that case (+4 for 32-bit targets). > > With this patch, the change on -g -dA is: > - .uleb128 0x4 # DW_AT_data_location > + .uleb128 0x2 # DW_AT_data_location > .byte 0x97 # DW_OP_push_object_address > - .byte 0x23 # DW_OP_plus_uconst > - .uleb128 0x8 > .byte 0x6 # DW_OP_deref > and gdb can debug the array in there again. > > Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? >
Yes. Thanks for looking at this! -- Steve
