We've decided to apply the following patch to ARM/embedded-6-branch.
Best regards,
Thomas
---------- Forwarded Message ----------
Subject: Re: [PATCH, ARM 4/7, ping1] Factor out MOVW/MOVT availability and
desirability checks
Date: Thursday 07 July 2016, 09:59:53
From: Thomas Preudhomme <thomas.preudho...@foss.arm.com>
To: Kyrill Tkachov <kyrylo.tkac...@foss.arm.com>
CC: gcc-patches@gcc.gnu.org, Richard Earnshaw <richard.earns...@arm.com>,
Ramana Radhakrishnan <ramana.radhakrish...@arm.com>
On Friday 20 May 2016 13:41:30 Kyrill Tkachov wrote:
> Hi Thomas,
>
> On 19/05/16 17:10, Thomas Preudhomme wrote:
> > On Wednesday 18 May 2016 11:47:47 Kyrill Tkachov wrote:
> >> Hi Thomas,
> >
> > Hi Kyrill,
> >
> > Please find below the updated patch and associated ChangeLog entry.
> >
> > *** gcc/ChangeLog ***
> >
> > 2016-05-18 Thomas Preud'homme <thomas.preudho...@arm.com>
> >
> > * config/arm/arm.h (TARGET_USE_MOVT): Check MOVT/MOVW
> > availability
> > with TARGET_HAVE_MOVT.
> > (TARGET_HAVE_MOVT): Define.
> > * config/arm/arm.c (const_ok_for_op): Check MOVT/MOVW
> > availability with TARGET_HAVE_MOVT.
> > * config/arm/arm.md (arm_movt): Use TARGET_HAVE_MOVT to check
> > movt
> > availability.
> > (addsi splitter): Use TARGET_THUMB && TARGET_HAVE_MOVT rather
> > than
> > TARGET_THUMB2.
> > (symbol_refs movsi splitter): Remove TARGET_32BIT check.
> > (arm_movtas_ze): Use TARGET_HAVE_MOVT to check movt availability.
> > * config/arm/constraints.md (define_constraint "j"): Use
> > TARGET_HAVE_MOVT to check movt availability.
>
> Please use capitalised MOVW/MOVT consistently in the ChangeLog.
> Ok with a fixed ChangeLog.
Committed with the following ChangeLog entry:
2016-05-18 Thomas Preud'homme <thomas.preudho...@arm.com>
* config/arm/arm.h (TARGET_USE_MOVT): Check MOVT/MOVW availability
with TARGET_HAVE_MOVT.
(TARGET_HAVE_MOVT): Define.
* config/arm/arm.c (const_ok_for_op): Check MOVT/MOVW
availability with TARGET_HAVE_MOVT.
* config/arm/arm.md (arm_movt): Use TARGET_HAVE_MOVT to check MOVT
availability.
(addsi splitter): Use TARGET_THUMB && TARGET_HAVE_MOVT rather than
TARGET_THUMB2.
(symbol_refs movsi splitter): Remove TARGET_32BIT check.
(arm_movtas_ze): Use TARGET_HAVE_MOVT to check MOVT availability.
* config/arm/constraints.md (define_constraint "j"): Use
TARGET_HAVE_MOVT to check MOVT availability.
Thanks.
Best regards,
Thomas
-----------------------------------------
