Re: cselib_record_set breaks due to auto_inc_dec

2012-12-24 Thread Paulo J. Matos
On 22/12/12 10:13, Alexandre Oliva wrote: On Dec 20, 2012, "Paulo Matos" wrote: This doesn't look sensible to me (but I might be overlooking a reason why we want to do so) in the context of cselib_record_sets, however, I think cselib_record_sets should instead have the patch applied: - for_e

Re: cselib_record_set breaks due to auto_inc_dec

2012-12-22 Thread Alexandre Oliva
On Dec 20, 2012, "Paulo Matos" wrote: > This doesn't look sensible to me (but I might be overlooking a reason why we > want to do so) in the context of cselib_record_sets, however, I think > cselib_record_sets should instead have the patch applied: > - for_each_inc_dec (&insn, cselib_record_aut

Re: Fwd: cselib_record_set breaks due to auto_inc_dec

2012-12-20 Thread Kenneth Zadeck
Subject: Re: Fwd: cselib_record_set breaks due to auto_inc_dec if i had to guess what was going on here, and since i do not have the example or even know the platform, i am just guessing, i would say that at some earlier pass, the sub expression in the regnote and the second set in the parallel were

RE: Fwd: cselib_record_set breaks due to auto_inc_dec

2012-12-20 Thread Paulo Matos
> -Original Message- > From: gcc-ow...@gcc.gnu.org [mailto:gcc-ow...@gcc.gnu.org] On Behalf Of > Kenneth Zadeck > Sent: 20 December 2012 12:46 > To: David Edelsohn; GCC Development > Subject: Re: Fwd: cselib_record_set breaks due to auto_inc_dec > > if i had to

Re: Fwd: cselib_record_set breaks due to auto_inc_dec

2012-12-20 Thread Kenneth Zadeck
set breaks due to auto_inc_dec To: "gcc@gcc.gnu.org" Hi, After pro and epilogue pass I have the following epilogue insn: (insn/f 20 19 21 2 (parallel [ (set (mem:DI (plus:SI (reg:SI 26 r26) (reg/f:SI 57 r57)) [0 S8 A8]) (reg:DI 57 r57

RE: cselib_record_set breaks due to auto_inc_dec

2012-12-20 Thread Paulo Matos
> -Original Message- > From: gcc-ow...@gcc.gnu.org [mailto:gcc-ow...@gcc.gnu.org] On Behalf Of Paulo > Matos > Sent: 19 December 2012 17:00 > To: gcc@gcc.gnu.org > Subject: cselib_record_set breaks due to auto_inc_dec > > Hi, > > After pro and epil

cselib_record_set breaks due to auto_inc_dec

2012-12-19 Thread Paulo Matos
Hi, After pro and epilogue pass I have the following epilogue insn: (insn/f 20 19 21 2 (parallel [ (set (mem:DI (plus:SI (reg:SI 26 r26) (reg/f:SI 57 r57)) [0 S8 A8]) (reg:DI 57 r57)) (set (reg/f:SI 57 r57) (plus:SI (r