On 24 October 2014 10:07, Marcus Shawcroft <marcus.shawcr...@gmail.com> wrote:
> On 21 October 2014 14:02, Christophe Lyon <christophe.l...@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.l...@linaro.org>
gcc.target/aarch64/advsimd-intrinsics/advsimd-intrinsics.exp: Skip
tests if target does not support Neon.
Christophe
> /Marcus
diff --git a/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/advsimd-intrinsics.exp b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/advsimd-intrinsics.exp
index 3aa0e1c..938086b 100644
--- a/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/advsimd-intrinsics.exp
+++ b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/advsimd-intrinsics.exp
@@ -32,6 +32,11 @@ load_lib torture-options.exp
dg-init
+if {[istarget arm*-*-*]
+ && ![check_effective_target_arm_neon_ok]} then {
+ return
+}
+
torture-init
set-torture-options $C_TORTURE_OPTIONS {{}} $LTO_TORTURE_OPTIONS
