[PATCH][ARM/AArch64 Testsuite] Cleanup advsimd-intrinsics.exp, removing unnecessary loop

Alan Lawrence alan.lawrence@arm.com
Fri May 29 11:46:00 GMT 2015


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}




More information about the Gcc-patches mailing list