Re: [PATCH][debug] Fix pre_dec handling in vartrack

2018-07-17 Thread Jakub Jelinek
On Tue, Jul 17, 2018 at 11:17:33AM +0200, Rainer Orth wrote: > it turns out the test FAILs on i386-pc-soalris2.11 with -m64: the dump > doesn't even contain argp: However, adding -fomit-frame-pointer makes > it work, and still PASSes on x86_64-pc-linux-gnu. > > Ok for for mainline? Ok, thanks.

Re: [PATCH][debug] Fix pre_dec handling in vartrack

2018-07-17 Thread Rainer Orth
Hi Jakub, > On Mon, Jul 16, 2018 at 09:24:10AM +0200, Jakub Jelinek wrote: >> On Sun, Jul 15, 2018 at 11:21:56PM +0200, Tom de Vries wrote: >> > 2018-07-15 Tom de Vries >> > >> >* var-tracking.c (vt_initialize): Fix pre_dec handling. Print adjusted >> >insn slim if dump_flags request

Re: [PATCH][debug] Fix pre_dec handling in vartrack

2018-07-16 Thread Jakub Jelinek
On Mon, Jul 16, 2018 at 09:24:10AM +0200, Jakub Jelinek wrote: > On Sun, Jul 15, 2018 at 11:21:56PM +0200, Tom de Vries wrote: > > 2018-07-15 Tom de Vries > > > > * var-tracking.c (vt_initialize): Fix pre_dec handling. Print adjusted > > insn slim if dump_flags request TDF_SLIM. > > >

Re: [PATCH][debug] Fix pre_dec handling in vartrack

2018-07-16 Thread Jakub Jelinek
On Sun, Jul 15, 2018 at 11:21:56PM +0200, Tom de Vries wrote: > 2018-07-15 Tom de Vries > > * var-tracking.c (vt_initialize): Fix pre_dec handling. Print adjusted > insn slim if dump_flags request TDF_SLIM. > > * gcc.target/i386/vartrack-1.c: New test. > > --- > --- a/gcc/v

[PATCH][debug] Fix pre_dec handling in vartrack

2018-07-15 Thread Tom de Vries
Hi, when compiling test-case gcc.target/i386/vartrack-1.c with -O1 -g, register bx is pushed in the prologue and popped in the epilogue: ... (insn/f 26 3 27 2 (set (mem:DI (pre_dec:DI (reg/f:DI 7 sp)) [0 S8 A8]) (reg:DI 3 bx)) "vartrack-1.c":10 61 {*pushdi2_rex64} (expr_list:REG_DE