On 5 June 2014 14:37, James Greenhalgh <james.greenha...@arm.com> wrote:
> 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  <vmaka...@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  <vmaka...@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
>
>

I did run the tests and saw several regressions. As I wasn't very
confident, I have relaunched my tests, they are still running.

Christophe.

Reply via email to