Re: [PATCH, GCC/testsuite/ARM] Add MOVT testing for ARMv8-M Baseline

[Kyrill Tkachov <kyrylo dot tkachov at foss dot arm dot com>]

On 22/05/17 08:48, Thomas Preudhomme wrote:
> Ping?
>
> Best regards,
>
> Thomas
>
> On 11/05/17 16:13, Thomas Preudhomme wrote:
> > Hi,
> >
> > When ARMv8-M Baseline support was introduced no testcase was added for
> > support of the MOVT instruction because GCC would always prefer a load
> > from literal pool. With the recent work to fix -mslow-flash-data by
> > Andre Simoes Dias Vieira and Prakhar Bahuguna it is now possible to
> > test that support.
> >
> > This commit adds a test for MOVT based on the MOVW test by using
> > -mslow-flash-data to ensure that GCC does not do a literal load of the
> > immediate. As such, this test is restricted to Cortex-M devices (where
> > -mslow-flash-data is available) with MOVT instruction.
> >
> > ChangeLog entry is as follows:
> >
> > *** gcc/testsuite/ChangeLog ***
> >
> > 2017-05-2017  Thomas Preud'homme <thomas.preudhomme@arm.com>
> >
> >     * gcc.target/arm/movsi_movt.c: New test.
> >     * gcc.target/arm/movdi_movt.c: New test.
> >
> > Testing: the new tests have been tested to pass for ARMv7-M and
> > ARMv8-M (Baseline and Mainline) on arm-none-eabi and armeb-none-eabi
> > targets.
> >
> > Is this ok for trunk?

Ok.
Thanks,
Kyrill

> > Best regards,
> >
> > Thomas