[20/46] Use require skip_aarch64_sve_tests

Message ID 20221217000818.3729389-21-tom@tromey.com
State Committed
Headers
Series Rewrite "require" test procedure and use it more often |

Commit Message

Tom Tromey Dec. 17, 2022, 12:07 a.m. UTC
  This changes some tests to use "require skip_aarch64_sve_tests".
---
 gdb/testsuite/gdb.arch/aarch64-sve.exp | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)
  

Comments

Luis Machado Dec. 19, 2022, 10:50 a.m. UTC | #1
On 12/17/22 00:07, Tom Tromey wrote:
> This changes some tests to use "require skip_aarch64_sve_tests".
> ---
>   gdb/testsuite/gdb.arch/aarch64-sve.exp | 5 +----
>   1 file changed, 1 insertion(+), 4 deletions(-)
> 
> diff --git a/gdb/testsuite/gdb.arch/aarch64-sve.exp b/gdb/testsuite/gdb.arch/aarch64-sve.exp
> index 803aae5fdfb..56793cec44e 100644
> --- a/gdb/testsuite/gdb.arch/aarch64-sve.exp
> +++ b/gdb/testsuite/gdb.arch/aarch64-sve.exp
> @@ -15,10 +15,7 @@
>   
>   # Test a binary that uses SVE and exercise changing the SVE vector length.
>   
> -if {[skip_aarch64_sve_tests]} {
> -    verbose "Skipping ${gdb_test_file_name}."
> -    return
> -}
> +require !skip_aarch64_sve_tests
>   
>   standard_testfile
>   if { [prepare_for_testing "failed to prepare" ${testfile} ${srcfile}] } {

LGTM.
  

Patch

diff --git a/gdb/testsuite/gdb.arch/aarch64-sve.exp b/gdb/testsuite/gdb.arch/aarch64-sve.exp
index 803aae5fdfb..56793cec44e 100644
--- a/gdb/testsuite/gdb.arch/aarch64-sve.exp
+++ b/gdb/testsuite/gdb.arch/aarch64-sve.exp
@@ -15,10 +15,7 @@ 
 
 # Test a binary that uses SVE and exercise changing the SVE vector length.
 
-if {[skip_aarch64_sve_tests]} {
-    verbose "Skipping ${gdb_test_file_name}."
-    return
-}
+require !skip_aarch64_sve_tests
 
 standard_testfile
 if { [prepare_for_testing "failed to prepare" ${testfile} ${srcfile}] } {