This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [Patch ARM-AArch64/testsuite v3 00/21] Neon intrinsics executable tests
- From: Ramana Radhakrishnan <ramana dot gcc at googlemail dot com>
- To: Christophe Lyon <christophe dot lyon at linaro dot org>
- Cc: Marcus Shawcroft <marcus dot shawcroft at gmail dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: Wed, 5 Nov 2014 13:15:54 +0000
- Subject: Re: [Patch ARM-AArch64/testsuite v3 00/21] Neon intrinsics executable tests
- Authentication-results: sourceware.org; auth=none
- References: <1413896593-26607-1-git-send-email-christophe dot lyon at linaro dot org> <CAFqB+PywOYtTurN+Vr3uzJbeYKr1yzWCj3efKRCDJ_bkO_7zbw at mail dot gmail dot com> <CAKdteOY6VTYVjZ3BC2Oe8872S5xpwyY5JJ2oFzjtGpj3_M=_Fg at mail dot gmail dot com>
- Reply-to: ramrad01 at arm dot com
On Sun, Oct 26, 2014 at 4:50 PM, Christophe Lyon
<christophe.lyon@linaro.org> wrote:
> On 24 October 2014 10:07, Marcus Shawcroft <marcus.shawcroft@gmail.com> wrote:
>> On 21 October 2014 14:02, Christophe Lyon <christophe.lyon@linaro.org> wrote:
>>> This patch series is an updated version of the series I sent here:
>>> https://gcc.gnu.org/ml/gcc-patches/2014-07/msg00022.html
>>>
>>> I addressed comments from Marcus and Richard, and decided to skip
>>> support for half-precision variants for the time being. I'll post
>>> dedicated patches later.
>>>
>>> Compared to v2:
>>> - the directory containing the new tests is named
>>> gcc.target/aarch64/adv-simd instead of
>>> gcc.target/aarch64/neon-intrinsics.
>>> - the driver is named adv-simd.exp instead of neon-intrinsics.exp
>>> - the driver is guarded against the new test parallelization framework
>>> - the README file uses 'Advanced SIMD (Neon)' instead of 'Neon'
>>>
>>
>> Thank you Christophe. Please commit all 21 patches in the series.
>>
> Thanks, I have committed the whole series.
>
> I've just realized afterwards that the tests aren't guarded against
> targets not supporting Neon.
>
> How about adding the attached small patch?
>
> (ChangeLog incorrectly formatted :-()
> 2014-10-26 Christophe Lyon <christophe.lyon@linaro.org>
>
> gcc.target/aarch64/advsimd-intrinsics/advsimd-intrinsics.exp: Skip
> tests if target does not support Neon.
Ok by me ...
Ramana
>
>
> Christophe
>
>> /Marcus