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: Jeff Law <law at redhat dot com>, 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: Wed, 11 Jan 2017 15:09:15 +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> <586F772B.50209@arm.com> <b47bfbc0-7a39-b72c-c6ef-a7e69729b383@redhat.com>
On 06/01/17 15:47, Jeff Law wrote:
> On 01/06/2017 03:53 AM, Andre Vieira (lists) wrote:
>> 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.
> Yes, that should be fine to backport to the active release branches.
>
> jeff
OK, I have committed the backports to gcc-5 and gcc-6 branches.
Cheers,
Andre