This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [ARM][testsuite] neon-testgen.ml removal


On 4 July 2016 at 18:28, Kyrill Tkachov <kyrylo.tkachov@foss.arm.com> wrote:
> Hi Christophe,
>
>
> On 22/06/16 16:52, Christophe Lyon wrote:
>>
>> Hi,
>>
>> This is a new attempt at removing neon-testgen.ml and generated files.
>>
>> Compared to my previous version several months ago:
>> - I have recently added testcases to make sure we do not lose coverage
>> as described in
>> https://gcc.gnu.org/ml/gcc-patches/2015-11/msg02922.html
>> - I now also remove neon.ml as requested by Kyrylo in
>> https://gcc.gnu.org/ml/gcc-patches/2015-11/msg01664.html, and moved
>> the remaining hand-written tests up to gcc.target/arm.
>>
>> Doing this, I had to slightly update vst1Q_laneu64-1.c because it's
>> now compiled with more pedantic flags and there was a signed/unsigned
>> char buffer pointer mismatch.
>>
>> Sorry, I had to compress the patch, otherwise it's too large and rejected
>> by the list server.
>>
>> OK?
>>
>> Christophe
>
>
>     [ARM] neon-testgen.ml, neon.ml and generated files removal.
>         gcc/
>     2016-06-17  Christophe Lyon<christophe.lyon@linaro.org>
>                 * config/arm/neon-testgen.ml: Delete.
>         * config/arm/neon.ml: Delete.
>         gcc/testsuite/
>     2016-06-17  Christophe Lyon<christophe.lyon@linaro.org>
>                 * gcc.target/arm/neon/polytypes.c: Move to ...
>         * gcc.target/arm/polytypes.c: ... here.
>         * gcc.target/arm/neon/pr51534.c: Move to ...
>         * gcc.target/arm/pr51534.c: ... here.
>         * gcc.target/arm/neon/vect-vcvt.c: Move to ...
>         * gcc.target/arm/vect-vcvt.c: ... here.
>         * gcc.target/arm/neon/vect-vcvtq.c: Move to ...
>         * gcc.target/arm/vect-vcvtq.c: ... here.
>         * gcc.target/arm/neon/vfp-shift-a2t2.c: Move to ...
>         * gcc.target/arm/vfp-shift-a2t2.c: ... here.
>         * gcc.target/arm/neon/vst1Q_laneu64-1.c: Move to ...
>         * gcc.target/arm/vst1Q_laneu64-1.c: ... here. Fix foo() prototype.
>         * gcc.target/arm/neon/neon.exp: Delete.
>         * gcc.target/arm/neon/*.c: Delete.
>
> I think this should be "* gcc.target/arm/neon/: Delete." to make it clear
> that the
> directory is being removed.
>
> This is ok for trunk.
> Thanks for dealing with this!
>
Thanks, committed as r238000 with the suggested ChangeLog change.

Christophe

> Kyrill
>
>


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]