This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][ARM/AArch64 Testsuite] Cleanup advsimd-intrinsics.exp, removing unnecessary loop
- From: Alan Lawrence <alan dot lawrence at arm dot com>
- To: Christophe Lyon <christophe dot lyon at linaro dot org>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, James Greenhalgh <James dot Greenhalgh at arm dot com>
- Date: Fri, 29 May 2015 11:48:34 +0100
- Subject: Re: [PATCH][ARM/AArch64 Testsuite] Cleanup advsimd-intrinsics.exp, removing unnecessary loop
- Authentication-results: sourceware.org; auth=none
- References: <55674646 dot 4060306 at arm dot com> <CAKdteOYNkEOb3N8r+0Cs6zqX24WHdJWvnEG+e1Q5m7C_oWnePA at mail dot gmail dot com>
Christophe Lyon wrote:
This looks OK, but why can't you also drop the other torture-related
lines as you did in your previous patch?
I mean:
load_lib c-torture.exp
load_lib torture-options.exp
etc...
We need c-torture.exp in order to set-torture-options; we need to
set-torture-options to get the right list of variants, as otherwise we lose "-Og
-g" (as I did in my original patch 13/14 for float16). However, I think we can
drop the rest - here's v2, which I've cross-tested on aarch64-none-elf,
aarch64_be-none-elf and arm-none-eabi:
diff --git a/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/advsimd-intrins
index 583832a..4679a73 100644
--- a/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/advsimd-intrinsics.exp
+++ b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/advsimd-intrinsics.exp
@@ -27,8 +27,6 @@ load_lib gcc-dg.exp
# Initialize `dg'.
load_lib c-torture.exp
-load_lib target-supports.exp
-load_lib torture-options.exp
dg-init
@@ -48,29 +46,15 @@ if {[istarget arm*-*-*]} then {
}
}
-torture-init
set-torture-options $C_TORTURE_OPTIONS {{}} $LTO_TORTURE_OPTIONS
# Make sure Neon flags are provided, if necessary.
set additional_flags [add_options_for_arm_neon ""]
# Main loop.
-foreach src [lsort [glob -nocomplain $srcdir/$subdir/*.c]] {
- # If we're only testing specific files and this isn't one of them, skip it.
- if ![runtest_file_p $runtests $src] then {
- continue
- }
-
- # runtest_file_p is already run above, and the code below can run
- # runtest_file_p again, make sure everything for this test is
- # performed if the above runtest_file_p decided this runtest
- # instance should execute the test
- gcc_parallel_test_enable 0
- gcc-dg-runtest $src "" $additional_flags
- gcc_parallel_test_enable 1
-}
+gcc-dg-runtest [lsort [glob -nocomplain $srcdir/$subdir/*.c]] \
+ "" ${additional_flags}
# All done.
set dg-do-what-default ${save-dg-do-what-default}