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:

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

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 robert.sucha...@imgtec.com * lra.c (lra): Set lra_in_progress before check_rtl call. * recog.c (insn_invalid_p): Add !lra_in_progress to prevent

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 robert.sucha...@imgtec.com * 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

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

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

2013-11-20 Thread Robert Suchanek
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 patch is OK, could someone commit the patch? I

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