Re: RFA: patch to fix PR59787 (arm target)

2014-01-14 Thread Yvan Roux
Thanks for the hint Vladimir, I'll pass some validation on arm.c and arm.md/aarch64.md separately. On 14 January 2014 20:09, Vladimir Makarov wrote: > On 01/14/2014 01:41 PM, Yvan Roux wrote: >>> A quick grep of the arm backend shows 11 instances of reload_in_progress: >>> >>> arm.c: && !(re

Re: RFA: patch to fix PR59787 (arm target)

2014-01-14 Thread Vladimir Makarov
On 01/14/2014 01:41 PM, Yvan Roux wrote: >> A quick grep of the arm backend shows 11 instances of reload_in_progress: >> >> arm.c: && !(reload_in_progress || reload_completed) >> arm.c: if (! (reload_in_progress || reload_completed) >> arm.c: if (! (reload_in_progress || reload_completed) >>

Re: RFA: patch to fix PR59787 (arm target)

2014-01-14 Thread Yvan Roux
> A quick grep of the arm backend shows 11 instances of reload_in_progress: > > arm.c: && !(reload_in_progress || reload_completed) > arm.c: if (! (reload_in_progress || reload_completed) > arm.c: if (! (reload_in_progress || reload_completed) > arm.c: if (! (reload_in_progress || reload_co

Re: RFA: patch to fix PR59787 (arm target)

2014-01-14 Thread Richard Earnshaw
On 14/01/14 16:48, Vladimir Makarov wrote: > The following patch fixes > > http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59787 > > The problem of LRA looping was in that address with soft frame pointer > was rejected as valid constraint in iwmmxt_amd_movdi insn. > > Ok to commit? > > 2014-01-

Re: RFA: patch to fix PR59787 (arm target)

2014-01-14 Thread Richard Earnshaw
On 14/01/14 16:48, Vladimir Makarov wrote: > The following patch fixes > > http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59787 > > The problem of LRA looping was in that address with soft frame pointer > was rejected as valid constraint in iwmmxt_amd_movdi insn. > > Ok to commit? > > 2014-01-

Re: RFA: patch to fix PR59787 (arm target)

2014-01-14 Thread Renlin Li
Hi Vladimir, I have test your patch with the following configurations, , and it indeed eliminates the ICE. --target=arm-none-linux-gnueabihf --with-arch=armv7-a --with-fpu=vfpv3-d16 --with-float=softfp Thank you! Kind regards, Renlin Li On 14/01/14 16:48, Vladimir Makarov wrote: The fol

RFA: patch to fix PR59787 (arm target)

2014-01-14 Thread Vladimir Makarov
The following patch fixes http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59787 The problem of LRA looping was in that address with soft frame pointer was rejected as valid constraint in iwmmxt_amd_movdi insn. Ok to commit? 2014-01-14 Vladimir Makarov PR target/59787 * confi