Re: patch to fix PR58784 (ARM LRA crash)

2013-11-08 Thread Steven Bosscher
On Fri, Nov 8, 2013 at 10:13 PM, Steven Bosscher wrote: > On Thu, Nov 7, 2013 at 10:42 AM, Yvan Roux wrote: > This store_minmaxsi insn is not recognized because > optimize_function_for_size_p() is false. The problem is in the place > where this insn was created. Trying to figure that out now... L

Re: patch to fix PR58784 (ARM LRA crash)

2013-11-08 Thread Steven Bosscher
On Thu, Nov 7, 2013 at 10:42 AM, Yvan Roux wrote: > Hi, > > I've tested LRA on ARM in several configuration and it occurs that > only one regression is observed in Fortran for targets A15 and A9 in > hard and soft fp amd Thumb mode enabled by default (ARMv5 is clean). > The failing test case is gf

Re: patch to fix PR58784 (ARM LRA crash)

2013-11-07 Thread Yvan Roux
Hi, I've tested LRA on ARM in several configuration and it occurs that only one regression is observed in Fortran for targets A15 and A9 in hard and soft fp amd Thumb mode enabled by default (ARMv5 is clean). The failing test case is gfortran.dg/realloc_on_assign_11.f90 in -O3 and the error seems

Re: patch to fix PR58784 (ARM LRA crash)

2013-10-31 Thread Yvan Roux
(this time in plain text !) > Does this mean we can now turn on LRA for ARM state by default and > start looking at performance issues if any ? With the other patch for 58785 and a small one I've to post, we are even about to turn LRA on by default completely, as the compiler now bootstrap also i

Re: patch to fix PR58784 (ARM LRA crash)

2013-10-31 Thread Ramana Radhakrishnan
On Wed, Oct 30, 2013 at 3:03 PM, Vladimir Makarov wrote: > The following patch fixes > > http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58784 > > LRA has an old check of legitimate addresses. It was written before a newer > address decomposition code which makes more correct checks of addresses. > >

Re: patch to fix PR58784 (ARM LRA crash)

2013-10-31 Thread Ramana Radhakrishnan
On Wed, Oct 30, 2013 at 3:03 PM, Vladimir Makarov wrote: > The following patch fixes > > http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58784 > > LRA has an old check of legitimate addresses. It was written before a newer > address decomposition code which makes more correct checks of addresses. > >

patch to fix PR58784 (ARM LRA crash)

2013-10-30 Thread Vladimir Makarov
The following patch fixes http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58784 LRA has an old check of legitimate addresses. It was written before a newer address decomposition code which makes more correct checks of addresses. So I removed the old check. Committed as rev. 204215. 2013-10-30