This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Make more use of subreg_lowpart_offset
- From: Jeff Law <law at redhat dot com>
- To: gcc-patches at gcc dot gnu dot org, richard dot sandiford at linaro dot org
- Date: Thu, 12 Oct 2017 00:26:12 -0600
- Subject: Re: Make more use of subreg_lowpart_offset
- Authentication-results: sourceware.org; auth=none
- Authentication-results: ext-mx08.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com
- Authentication-results: ext-mx08.extmail.prod.ext.phx2.redhat.com; spf=fail smtp.mailfrom=law at redhat dot com
- Dmarc-filter: OpenDMARC Filter v1.3.2 mx1.redhat.com D367AC059B72
- References: <871so2v9ov.fsf@linaro.org>
On 08/23/2017 04:51 AM, Richard Sandiford wrote:
> This patch uses subreg_lowpart_offset in places that open-coded
> the calculation. It also uses it in regcprop.c to test whether,
> after a mode change, the first register in a multi-register group
> is still the right one.
>
> Tested on aarch64-linux-gnu and x86_64-linux-gnu, and by making sure
> that there were no differences in testsuite assembly output for one
> target per CPU. OK to install?
>
> Richard
>
>
> 2017-08-23 Richard Sandiford <richard.sandiford@linaro.org>
> Alan Hayward <alan.hayward@arm.com>
> David Sherwood <david.sherwood@arm.com>
>
> gcc/
> * calls.c (expand_call): Use subreg_lowpart_offset.
> * cse.c (cse_insn): Likewise.
> * regcprop.c (copy_value): Likewise.
> (copyprop_hardreg_forward_1): Likewise.
It was far from obvious that the open coded versions actually
corresponded to subreg_lowpart_offset. Sorry it took so long.
OK.
jeff