This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][AARCH64]Replace gen_rtx_PLUS with plus_constant
- From: James Greenhalgh <james dot greenhalgh at arm dot com>
- To: Renlin Li <Renlin dot Li at arm dot com>
- Cc: Marcus Shawcroft <marcus dot shawcroft at gmail dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: Mon, 23 Sep 2013 09:26:23 +0100
- Subject: Re: [PATCH][AARCH64]Replace gen_rtx_PLUS with plus_constant
- Authentication-results: sourceware.org; auth=none
- References: <523C593E dot 9060706 at arm dot com> <CAFqB+PyijEMsxMBDtox1HDn8MqbQSyVjeWNprU4WufwVF0VbUw at mail dot gmail dot com> <523C5E7B dot 2010203 at arm dot com> <20130921083402 dot GA11608 at arm dot com>
On Sat, Sep 21, 2013 at 09:34:34AM +0100, James Greenhalgh wrote:
> On Fri, Sep 20, 2013 at 03:40:59PM +0100, Renlin Li wrote:
> > Thank you, can you please commit it for me?
> >
> > Kind regards,
> > Renlin Li
> >
> > On 09/20/13 15:26, Marcus Shawcroft wrote:
> > > On 20 September 2013 15:18, Renlin Li <renli.li@arm.com> wrote:
> > >
> > >> 2013-09-20 Renlin Li <renlin.li@arm.com>
> > >>
> > >> * config/aarch64/aarch64.c (aarch64_expand_prologue): Use plus_constant.
> > >> (aarch64_expand_epilogue): Likewise.
> > >> (aarch64_legitimize_reload_address): Likewise.
>
> Hi Renlin,
>
> This patch appears to have caused a number of regressions on
> an aarch64-none-elf test run.
Marcus, Renlin,
I'm also seeing issues building an AArch64 Linux toolchain with this patch
applied, so I've reverted it for now.
Thanks,
James