This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] PR78255: Make postreload aware of NO_FUNCTION_CSE
- From: "Andre Vieira (lists)" <Andre dot SimoesDiasVieira at arm dot com>
- To: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Cc: Jakub Jelinek <jakub at redhat dot com>, Richard Biener <richard dot guenther at gmail dot com>
- Date: Fri, 6 Jan 2017 10:53:31 +0000
- Subject: Re: [PATCH] PR78255: Make postreload aware of NO_FUNCTION_CSE
- Authentication-results: sourceware.org; auth=none
- References: <584AB9AA.6030800@arm.com> <334ff580-3e7d-22fb-83da-da18acd84244@redhat.com> <584ACF02.9070101@arm.com> <dcec00b3-d46b-529b-84b5-bb661a0f811d@redhat.com> <CAJA7tRZ93zT5Q2rkDzhqLpJLEhNbmRTiMZOivHegPT1+AJj__A@mail.gmail.com> <584AD8E9.3000406@arm.com> <db6385c8-287f-03db-f373-d955b18995fa@redhat.com>
On 09/12/16 16:31, Bernd Schmidt wrote:
> On 12/09/2016 05:16 PM, Andre Vieira (lists) wrote:
>
>> Regardless, 'reload_cse_simplify' would never perform the opposite
>> transformation. It checks whether it can replace anything within the
>> first argument INSN, with the second argument TESTREG. As the name
>> implies this will always be a register. I double checked, the function
>> is only called in 'reload_cse_regs' and 'testreg' is created using
>> 'gen_rtx_REG'.
>
> Ok, let's go ahead with it.
>
>
> Bernd
>
Hello,
Is it OK to backport this (including the testcase fix) to gcc-6-branch?
Patches apply cleanly and full bootstrap and regression tests for
aarch64- and arm-none-linux-gnueabihf. Regression tested for arm-none-eabi.
Cheers,
Andre