This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, GCC/ARM, Stage 1] Add missing TARGET_32BIT conditional to movsi
- From: Kyrill Tkachov <kyrylo dot tkachov at foss dot arm dot com>
- To: Prakhar Bahuguna <prakhar dot bahuguna at arm dot com>, gcc-patches at gcc dot gnu dot org
- Cc: nd at arm dot com, Richard dot Earnshaw at arm dot com, Ramana dot Radhakrishnan at arm dot com
- Date: Thu, 11 May 2017 10:58:52 +0100
- Subject: Re: [PATCH, GCC/ARM, Stage 1] Add missing TARGET_32BIT conditional to movsi
- Authentication-results: sourceware.org; auth=none
- References: <20170511095610.bbtbhcx4idax63so@e107464-lin.cambridge.arm.com>
On 11/05/17 10:56, Prakhar Bahuguna wrote:
Resolve the regressions introduced on non-Thumb targets by the Purecode for
ARMv8-M Baseline patch. The TARGET_32BIT conditional has been re-added to the
movsi expander and splitter in addition to TARGET_HAVE_MOVT.
gcc/ChangeLog:
2017-05-11 Prakhar Bahuguna <prakhar.bahuguna@arm.com>
* config/arm/arm.md (movsi): Add TARGET_32BIT in addition to the
TARGET_HAVE_MOVT conditional.
(movt splitter): Likewise.
Testing done: Full regression testing for ARMv5 (including XScale), ARMv7-A,
ARMv7-M and ARMv8-M. The failing tests in particular (gcc.target/arm/scd42-2.c
and g++.dg/torture/vshuf-v4si.c) now pass and no further regressions were
found.
Ok if a bootstrap on arm-none-linux-gnueabihf target passes as well.
Thanks,
Kyrill