This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: patch to fix PR61325


On Wed, Jun 04, 2014 at 08:00:51PM +0100, Vladimir Makarov wrote:
> On 2014-06-03, 6:02 PM, James Greenhalgh wrote:
> > On Thu, May 29, 2014 at 06:38:22PM +0100, Vladimir Makarov wrote:
> >>    The following patch PR61325.  The details can be found on
> >>
> >> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61325
> >>
> >>    The patch was bootstrapped and tested on x86/x86-64.
> >>
> >>    Committed as rev. 211060 to gcc-4.9 branch and as rev.211061 to trunk.
> >>
> >> 2014-05-29  Vladimir Makarov  <vmakarov@redhat.com>
> >>
> >>          PR rtl-optimization/61325
> >>          * lra-constraints.c (process_address): Rename to
> >>          process_address_1.
> >>          (process_address): New function.
> >>
> >> 2014-05-29  Vladimir Makarov  <vmakarov@redhat.com>
> >>
> >>          PR rtl-optimization/61325
> >>          * gcc.target/aarch64/pr61325.c: New.
> >
> > Hi Vlad,
> >
> > This patch appears to cause issues where the ARM backend can get stuck in a
> > seemingly infinite loop.
> >
> > Compiling:
> >
> > ./gcc.c-torture/compile/unalign-1.c
> >
> 
> Sorry for inconvenience.
> 
> Could you test the following patch

Hi Vlad,

The patch works for me. I've bootstrapped and tested it on
arm-none-linux-gnueabihf. It has also had a regression run on aarch64-none-elf.
Cristophe, did you get a chance to do any more thorough testing of the patch?

>From my perspective, I think this should go in, and be backported to 4.9.

Thanks,
James

 


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]