This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, LRA] Never reload fixed form constraints memory operand
- From: Vladimir Makarov <vmakarov at redhat dot com>
- To: Thomas Preudhomme <thomas dot preudhomme at linaro dot org>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Wed, 3 Oct 2018 23:12:20 -0400
- Subject: Re: [PATCH, LRA] Never reload fixed form constraints memory operand
- References: <CAKnkMGvXi16uEfXRfHe2WPHkoCMLsoCBmHm8op=FmAHcN3nLkg@mail.gmail.com>
On 10/03/2018 12:47 PM, Thomas Preudhomme wrote:
Best regards,
Thomas
never_reload_fixed_address_operand.patch
From 2831d8b886d92513c2d30d43a6a989d2bbd0ceee Mon Sep 17 00:00:00 2001
From: Thomas Preud'homme<thomas.preudhomme@linaro.org>
Date: Thu, 27 Sep 2018 09:50:12 +0100
Subject: [PATCH] [PATCH, LRA] Never reload fixed form constraints memory
operand
Hi,
The unconditional reload of address operand for recognized instruction
in process_address_1 prevent the patch for fixing "PR85434: Address of
stack protector guard spilled to stack on ARM" proposed at [1]. The code
in this patch attempt to control which registers are used to make PIC
access but the reload performed by process_address_1 will use generic
PIC access. This patch removes the test for the instruction to be
unrecognized to do the reload, thus always avoiding to reload address
operand for fixed constraints (such as "X" used in the patch).
[1]https://gcc.gnu.org/ml/gcc-patches/2018-08/msg01838.html
ChangeLog entry is as follows:
*** gcc/ChangeLog ***
2018-10-03 Thomas Preud'homme<thomas.preudhomme@linaro.org>
* lra-constraints.c (process_address_1): Bail out for all
satisfied fixed constraints.
Testing: Successfully bootstrapped and regtested on:
- arm-linux-gnueabihf (both Arm and Thumb2 mode)
- aarch64-linux-gnu
- x86_64-linux-gnu
- i386-linux-gnu
- sparc64-linux-gnu (gcc202)
- powerpc64le-linux-gnu (gcc112)
Is this ok for trunk?
OK. Thank you for testing all these targets, Thomas.