Re: [PING][PATCH] LRA: check_rtl modifies RTL instruction stream

2013-12-03 Thread Alan Modra
On Mon, Dec 02, 2013 at 11:04:39PM -0700, Jeff Law wrote: > On 11/28/13 16:50, Alan Modra wrote: > > > >This is due to that innocuous seeming change of setting > >lra_in_progress before calling check_rtl(), in combination with > >previous changes Vlad made to the rs6000 backend here: > >http://gcc.

Re: [PING][PATCH] LRA: check_rtl modifies RTL instruction stream

2013-12-02 Thread Jeff Law
On 11/28/13 16:50, Alan Modra wrote: This is due to that innocuous seeming change of setting lra_in_progress before calling check_rtl(), in combination with previous changes Vlad made to the rs6000 backend here: http://gcc.gnu.org/ml/gcc-patches/2013-10/msg02208.html In particular the "Call legi

Re: [PING][PATCH] LRA: check_rtl modifies RTL instruction stream

2013-11-29 Thread Vladimir Makarov
On 11/28/2013, 6:50 PM, Alan Modra wrote: On Wed, Nov 20, 2013 at 11:18:49AM -0700, Jeff Law wrote: 2013-11-13 Robert Suchanek * lra.c (lra): Set lra_in_progress before check_rtl call. * recog.c (insn_invalid_p): Add !lra_in_progress to prevent adding clobber regs whe

Re: [PING][PATCH] LRA: check_rtl modifies RTL instruction stream

2013-11-28 Thread Alan Modra
On Wed, Nov 20, 2013 at 11:18:49AM -0700, Jeff Law wrote: > >2013-11-13 Robert Suchanek > > > > * lra.c (lra): Set lra_in_progress before check_rtl call. > > * recog.c (insn_invalid_p): Add !lra_in_progress to prevent > > adding clobber regs when LRA is runnin

RE: [PING][PATCH] LRA: check_rtl modifies RTL instruction stream

2013-11-21 Thread Robert Suchanek
>Thanks. Vlad may not be available right now, and even if he is, he's >probably typing one-handed. > >So I took care of installing this for you. > >Thanks, >Jeff Thanks! Regards, Robert

Re: [PING][PATCH] LRA: check_rtl modifies RTL instruction stream

2013-11-20 Thread Jeff Law
On 11/20/13 10:27, Robert Suchanek wrote: Hi, I'm pinging for the review of a patch. Original post: http://gcc.gnu.org/ml/gcc-patches/2013-11/msg01608.html The attached patch is slightly modified as the indentation was incorrect and realised that after sending the email last time. If the patc