From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 2126) id 4CD463858425; Fri, 13 Jan 2023 20:38:40 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 4CD463858425 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1673642320; bh=4pjPWUx/711ibqLOfJO1IaLMgoztcQWG6fqL1XiZSo0=; h=From:To:Subject:Date:From; b=FjeYXxlnRGVhBV1uPhcKx5YMEWMBvXI2b45qtn37678jjLzM0nx5weZ6pCkcbgW3e UAejESmueAGfZDKTwopogCxnvnLGq4fgRRrwERwWHi4vSRQ+yrUoNoP4VOahSua1wo RyDrGtu01+gNyFbxfIIGtcxo0ghhVNPJZdKYzxWw= Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable From: Tom Tromey To: gdb-cvs@sourceware.org Subject: [binutils-gdb] Rename to allow_btrace_tests X-Act-Checkin: binutils-gdb X-Git-Author: Tom Tromey X-Git-Refname: refs/heads/master X-Git-Oldrev: d1821835860e1f11e17ac5de5109d225a736cda0 X-Git-Newrev: 1ed844ca1e581f80b3ecdcc99a8dab11034e684d Message-Id: <20230113203840.4CD463858425@sourceware.org> Date: Fri, 13 Jan 2023 20:38:40 +0000 (GMT) List-Id: https://sourceware.org/git/gitweb.cgi?p=3Dbinutils-gdb.git;h=3D1ed844ca1e58= 1f80b3ecdcc99a8dab11034e684d commit 1ed844ca1e581f80b3ecdcc99a8dab11034e684d Author: Tom Tromey Date: Sun Jan 8 10:44:15 2023 -0700 Rename to allow_btrace_tests =20 This changes skip_btrace_tests to invert the sense, and renames it to allow_btrace_tests. Diff: --- gdb/testsuite/gdb.btrace/buffer-size.exp | 2 +- gdb/testsuite/gdb.btrace/data.exp | 2 +- gdb/testsuite/gdb.btrace/delta.exp | 2 +- gdb/testsuite/gdb.btrace/dlopen.exp | 2 +- gdb/testsuite/gdb.btrace/enable-new-thread.exp | 2 +- gdb/testsuite/gdb.btrace/enable-running.exp | 2 +- gdb/testsuite/gdb.btrace/enable.exp | 2 +- gdb/testsuite/gdb.btrace/exception.exp | 2 +- gdb/testsuite/gdb.btrace/function_call_history.exp | 2 +- gdb/testsuite/gdb.btrace/gcore.exp | 2 +- gdb/testsuite/gdb.btrace/instruction_history.exp | 2 +- gdb/testsuite/gdb.btrace/multi-inferior.exp | 2 +- gdb/testsuite/gdb.btrace/multi-thread-step.exp | 2 +- gdb/testsuite/gdb.btrace/nohist.exp | 2 +- gdb/testsuite/gdb.btrace/non-stop.exp | 2 +- gdb/testsuite/gdb.btrace/reconnect.exp | 2 +- gdb/testsuite/gdb.btrace/record_goto-step.exp | 2 +- gdb/testsuite/gdb.btrace/record_goto.exp | 2 +- gdb/testsuite/gdb.btrace/rn-dl-bind.exp | 2 +- gdb/testsuite/gdb.btrace/segv.exp | 2 +- gdb/testsuite/gdb.btrace/step.exp | 2 +- gdb/testsuite/gdb.btrace/stepi.exp | 2 +- gdb/testsuite/gdb.btrace/tailcall-only.exp | 2 +- gdb/testsuite/gdb.btrace/tailcall.exp | 2 +- gdb/testsuite/gdb.btrace/unknown_functions.exp | 2 +- gdb/testsuite/gdb.btrace/vdso.exp | 2 +- .../gdb.python/py-record-btrace-threads.exp | 2 +- gdb/testsuite/gdb.python/py-record-btrace.exp | 2 +- gdb/testsuite/lib/gdb.exp | 30 +++++++++++-------= ---- 29 files changed, 43 insertions(+), 43 deletions(-) diff --git a/gdb/testsuite/gdb.btrace/buffer-size.exp b/gdb/testsuite/gdb.b= trace/buffer-size.exp index d0a9086604f..e632ba56cf9 100644 --- a/gdb/testsuite/gdb.btrace/buffer-size.exp +++ b/gdb/testsuite/gdb.btrace/buffer-size.exp @@ -17,7 +17,7 @@ # You should have received a copy of the GNU General Public License # along with this program. If not, see . =20 -require !skip_btrace_tests +require allow_btrace_tests =20 standard_testfile record_goto.c if [prepare_for_testing "failed to prepare" $testfile $srcfile] { diff --git a/gdb/testsuite/gdb.btrace/data.exp b/gdb/testsuite/gdb.btrace/d= ata.exp index 8be2a4ccdea..67d4b094a8d 100644 --- a/gdb/testsuite/gdb.btrace/data.exp +++ b/gdb/testsuite/gdb.btrace/data.exp @@ -17,7 +17,7 @@ # You should have received a copy of the GNU General Public License # along with this program. If not, see . =20 -require !skip_btrace_tests +require allow_btrace_tests =20 standard_testfile if [prepare_for_testing "failed to prepare" $testfile $srcfile] { diff --git a/gdb/testsuite/gdb.btrace/delta.exp b/gdb/testsuite/gdb.btrace/= delta.exp index e4307e15a5b..f671937d260 100644 --- a/gdb/testsuite/gdb.btrace/delta.exp +++ b/gdb/testsuite/gdb.btrace/delta.exp @@ -17,7 +17,7 @@ # You should have received a copy of the GNU General Public License # along with this program. If not, see . =20 -require !skip_btrace_tests +require allow_btrace_tests =20 standard_testfile record_goto.c if [prepare_for_testing "failed to prepare" $testfile $srcfile] { diff --git a/gdb/testsuite/gdb.btrace/dlopen.exp b/gdb/testsuite/gdb.btrace= /dlopen.exp index b742cfa86bd..8cd0b9db9aa 100644 --- a/gdb/testsuite/gdb.btrace/dlopen.exp +++ b/gdb/testsuite/gdb.btrace/dlopen.exp @@ -15,7 +15,7 @@ # You should have received a copy of the GNU General Public License # along with this program. If not, see . =20 -require !skip_btrace_tests !skip_shlib_tests +require allow_btrace_tests !skip_shlib_tests =20 standard_testfile =20 diff --git a/gdb/testsuite/gdb.btrace/enable-new-thread.exp b/gdb/testsuite= /gdb.btrace/enable-new-thread.exp index 22cc33f142f..a9c7d579e8b 100644 --- a/gdb/testsuite/gdb.btrace/enable-new-thread.exp +++ b/gdb/testsuite/gdb.btrace/enable-new-thread.exp @@ -17,7 +17,7 @@ =20 # Test that new threads of recorded inferiors also get recorded. =20 -require !skip_btrace_tests +require allow_btrace_tests =20 standard_testfile if [prepare_for_testing "failed to prepare" $testfile $srcfile {debug pthr= eads}] { diff --git a/gdb/testsuite/gdb.btrace/enable-running.exp b/gdb/testsuite/gd= b.btrace/enable-running.exp index 6e626e3a053..9dea21f5dfd 100644 --- a/gdb/testsuite/gdb.btrace/enable-running.exp +++ b/gdb/testsuite/gdb.btrace/enable-running.exp @@ -15,7 +15,7 @@ # You should have received a copy of the GNU General Public License # along with this program. If not, see . =20 -require !skip_btrace_tests +require allow_btrace_tests =20 standard_testfile if {[gdb_compile_pthreads "$srcdir/$subdir/$srcfile" "$binfile" executable= {debug}] !=3D "" } { diff --git a/gdb/testsuite/gdb.btrace/enable.exp b/gdb/testsuite/gdb.btrace= /enable.exp index bb57f7b8a74..acad4c6e828 100644 --- a/gdb/testsuite/gdb.btrace/enable.exp +++ b/gdb/testsuite/gdb.btrace/enable.exp @@ -17,7 +17,7 @@ # You should have received a copy of the GNU General Public License # along with this program. If not, see . =20 -require !skip_btrace_tests +require allow_btrace_tests =20 # start fresh - without an executable gdb_exit diff --git a/gdb/testsuite/gdb.btrace/exception.exp b/gdb/testsuite/gdb.btr= ace/exception.exp index e84ff14d193..9f12abab323 100755 --- a/gdb/testsuite/gdb.btrace/exception.exp +++ b/gdb/testsuite/gdb.btrace/exception.exp @@ -17,7 +17,7 @@ # You should have received a copy of the GNU General Public License # along with this program. If not, see . =20 -require !skip_btrace_tests +require allow_btrace_tests =20 # We expect a specific function call history. This gets messed up with # PIE on 32-bit. diff --git a/gdb/testsuite/gdb.btrace/function_call_history.exp b/gdb/tests= uite/gdb.btrace/function_call_history.exp index fd35d762796..e71e48fc0ca 100644 --- a/gdb/testsuite/gdb.btrace/function_call_history.exp +++ b/gdb/testsuite/gdb.btrace/function_call_history.exp @@ -17,7 +17,7 @@ # You should have received a copy of the GNU General Public License # along with this program. If not, see . =20 -require !skip_btrace_tests +require allow_btrace_tests =20 # We expect a specific function call history. This gets messed up with # PIE on 32-bit. diff --git a/gdb/testsuite/gdb.btrace/gcore.exp b/gdb/testsuite/gdb.btrace/= gcore.exp index 1e98860aee3..d18bda53469 100644 --- a/gdb/testsuite/gdb.btrace/gcore.exp +++ b/gdb/testsuite/gdb.btrace/gcore.exp @@ -17,7 +17,7 @@ # You should have received a copy of the GNU General Public License # along with this program. If not, see . =20 -require !skip_btrace_tests +require allow_btrace_tests =20 standard_testfile record_goto.c if [prepare_for_testing "failed to prepare" $testfile $srcfile] { diff --git a/gdb/testsuite/gdb.btrace/instruction_history.exp b/gdb/testsui= te/gdb.btrace/instruction_history.exp index 0e525c73cb7..35b43d68434 100644 --- a/gdb/testsuite/gdb.btrace/instruction_history.exp +++ b/gdb/testsuite/gdb.btrace/instruction_history.exp @@ -17,7 +17,7 @@ # You should have received a copy of the GNU General Public License # along with this program. If not, see . =20 -require !skip_btrace_tests +require allow_btrace_tests =20 standard_testfile .c .S if [prepare_for_testing "failed to prepare" $testfile "$srcfile $srcfile2"= {debug}] { diff --git a/gdb/testsuite/gdb.btrace/multi-inferior.exp b/gdb/testsuite/gd= b.btrace/multi-inferior.exp index bbc03daaccc..79e5a1c39bf 100644 --- a/gdb/testsuite/gdb.btrace/multi-inferior.exp +++ b/gdb/testsuite/gdb.btrace/multi-inferior.exp @@ -22,7 +22,7 @@ # # Each inferior can be recorded separately. =20 -require !skip_btrace_tests +require allow_btrace_tests =20 require !use_gdb_stub =20 diff --git a/gdb/testsuite/gdb.btrace/multi-thread-step.exp b/gdb/testsuite= /gdb.btrace/multi-thread-step.exp index a18734843ae..af61e5e73da 100644 --- a/gdb/testsuite/gdb.btrace/multi-thread-step.exp +++ b/gdb/testsuite/gdb.btrace/multi-thread-step.exp @@ -17,7 +17,7 @@ # You should have received a copy of the GNU General Public License # along with this program. If not, see . =20 -require !skip_btrace_tests +require allow_btrace_tests =20 standard_testfile if {[gdb_compile_pthreads "$srcdir/$subdir/$srcfile" "$binfile" executable= {debug}] !=3D "" } { diff --git a/gdb/testsuite/gdb.btrace/nohist.exp b/gdb/testsuite/gdb.btrace= /nohist.exp index 0875b8a6e22..f4ab977ded8 100644 --- a/gdb/testsuite/gdb.btrace/nohist.exp +++ b/gdb/testsuite/gdb.btrace/nohist.exp @@ -17,7 +17,7 @@ # You should have received a copy of the GNU General Public License # along with this program. If not, see . =20 -require !skip_btrace_tests +require allow_btrace_tests =20 standard_testfile record_goto.c if [prepare_for_testing "failed to prepare" $testfile $srcfile] { diff --git a/gdb/testsuite/gdb.btrace/non-stop.exp b/gdb/testsuite/gdb.btra= ce/non-stop.exp index 278064fbac5..530fafd9573 100644 --- a/gdb/testsuite/gdb.btrace/non-stop.exp +++ b/gdb/testsuite/gdb.btrace/non-stop.exp @@ -15,7 +15,7 @@ # You should have received a copy of the GNU General Public License # along with this program. If not, see . =20 -require !skip_btrace_tests +require allow_btrace_tests =20 standard_testfile if {[gdb_compile_pthreads "$srcdir/$subdir/$srcfile" "$binfile" executable= {debug}] !=3D "" } { diff --git a/gdb/testsuite/gdb.btrace/reconnect.exp b/gdb/testsuite/gdb.btr= ace/reconnect.exp index 548aa16ed29..1da9f4be566 100644 --- a/gdb/testsuite/gdb.btrace/reconnect.exp +++ b/gdb/testsuite/gdb.btrace/reconnect.exp @@ -19,7 +19,7 @@ =20 load_lib gdbserver-support.exp =20 -require !skip_btrace_tests +require allow_btrace_tests require !skip_gdbserver_tests =20 standard_testfile diff --git a/gdb/testsuite/gdb.btrace/record_goto-step.exp b/gdb/testsuite/= gdb.btrace/record_goto-step.exp index 1f289b47f64..35731c72b40 100644 --- a/gdb/testsuite/gdb.btrace/record_goto-step.exp +++ b/gdb/testsuite/gdb.btrace/record_goto-step.exp @@ -17,7 +17,7 @@ # You should have received a copy of the GNU General Public License # along with this program. If not, see . =20 -require !skip_btrace_tests +require allow_btrace_tests =20 standard_testfile record_goto.c if [prepare_for_testing "failed to prepare" $testfile $srcfile] { diff --git a/gdb/testsuite/gdb.btrace/record_goto.exp b/gdb/testsuite/gdb.b= trace/record_goto.exp index dce9ca18c6c..240e18f67d6 100644 --- a/gdb/testsuite/gdb.btrace/record_goto.exp +++ b/gdb/testsuite/gdb.btrace/record_goto.exp @@ -17,7 +17,7 @@ # You should have received a copy of the GNU General Public License # along with this program. If not, see . =20 -require !skip_btrace_tests +require allow_btrace_tests =20 # The "record goto" command jumps to a specific instruction in the executi= on # trace. To guarantee that we always get the same execution trace, we use diff --git a/gdb/testsuite/gdb.btrace/rn-dl-bind.exp b/gdb/testsuite/gdb.bt= race/rn-dl-bind.exp index f61aa936a8d..4a419a84fbb 100644 --- a/gdb/testsuite/gdb.btrace/rn-dl-bind.exp +++ b/gdb/testsuite/gdb.btrace/rn-dl-bind.exp @@ -21,7 +21,7 @@ # Test that we can reverse-next over the dynamic linker's symbol # lookup code. =20 -require !skip_btrace_tests +require allow_btrace_tests =20 standard_testfile if [prepare_for_testing "failed to prepare" $testfile $srcfile \ diff --git a/gdb/testsuite/gdb.btrace/segv.exp b/gdb/testsuite/gdb.btrace/s= egv.exp index 9d256795729..dfd04b3cae7 100644 --- a/gdb/testsuite/gdb.btrace/segv.exp +++ b/gdb/testsuite/gdb.btrace/segv.exp @@ -17,7 +17,7 @@ # You should have received a copy of the GNU General Public License # along with this program. If not, see . =20 -require !skip_btrace_tests +require allow_btrace_tests =20 standard_testfile if [prepare_for_testing "failed to prepare" $testfile $srcfile] { diff --git a/gdb/testsuite/gdb.btrace/step.exp b/gdb/testsuite/gdb.btrace/s= tep.exp index 8ad58d48fdc..219775cdeb1 100644 --- a/gdb/testsuite/gdb.btrace/step.exp +++ b/gdb/testsuite/gdb.btrace/step.exp @@ -17,7 +17,7 @@ # You should have received a copy of the GNU General Public License # along with this program. If not, see . =20 -require !skip_btrace_tests +require allow_btrace_tests =20 standard_testfile record_goto.c if [prepare_for_testing "failed to prepare" $testfile $srcfile] { diff --git a/gdb/testsuite/gdb.btrace/stepi.exp b/gdb/testsuite/gdb.btrace/= stepi.exp index f659d58cf7c..4592309796e 100644 --- a/gdb/testsuite/gdb.btrace/stepi.exp +++ b/gdb/testsuite/gdb.btrace/stepi.exp @@ -17,7 +17,7 @@ # You should have received a copy of the GNU General Public License # along with this program. If not, see . =20 -require !skip_btrace_tests +require allow_btrace_tests =20 # This test is stepping on instruction level. To guarantee that we always # get the same execution trace, we use an assembly source file. diff --git a/gdb/testsuite/gdb.btrace/tailcall-only.exp b/gdb/testsuite/gdb= .btrace/tailcall-only.exp index 16541b2eacf..857cc375ba3 100644 --- a/gdb/testsuite/gdb.btrace/tailcall-only.exp +++ b/gdb/testsuite/gdb.btrace/tailcall-only.exp @@ -20,7 +20,7 @@ # calls. This used to cause a crash in get_frame_type. # =20 -require !skip_btrace_tests +require allow_btrace_tests =20 # This test requires the compiler to generate a tail call. To guarantee t= hat # we always get one, we use an assembly source file. diff --git a/gdb/testsuite/gdb.btrace/tailcall.exp b/gdb/testsuite/gdb.btra= ce/tailcall.exp index f422fbb1b2a..7fbcd40c077 100644 --- a/gdb/testsuite/gdb.btrace/tailcall.exp +++ b/gdb/testsuite/gdb.btrace/tailcall.exp @@ -17,7 +17,7 @@ # You should have received a copy of the GNU General Public License # along with this program. If not, see . =20 -require !skip_btrace_tests +require allow_btrace_tests =20 # This test requires the compiler to generate a tail call. To guarantee t= hat # we always get one, we use an assembly source file. diff --git a/gdb/testsuite/gdb.btrace/unknown_functions.exp b/gdb/testsuite= /gdb.btrace/unknown_functions.exp index 291d7df5963..fadfabd57b8 100644 --- a/gdb/testsuite/gdb.btrace/unknown_functions.exp +++ b/gdb/testsuite/gdb.btrace/unknown_functions.exp @@ -17,7 +17,7 @@ # You should have received a copy of the GNU General Public License # along with this program. If not, see . =20 -require !skip_btrace_tests +require allow_btrace_tests =20 standard_testfile =20 diff --git a/gdb/testsuite/gdb.btrace/vdso.exp b/gdb/testsuite/gdb.btrace/v= dso.exp index 16a7e230786..2bfc476b3cc 100644 --- a/gdb/testsuite/gdb.btrace/vdso.exp +++ b/gdb/testsuite/gdb.btrace/vdso.exp @@ -20,7 +20,7 @@ # # Test that we can access the vdso memory during replay for stepping. =20 -require !skip_btrace_tests +require allow_btrace_tests =20 standard_testfile if [prepare_for_testing "failed to prepare" $testfile $srcfile] { diff --git a/gdb/testsuite/gdb.python/py-record-btrace-threads.exp b/gdb/te= stsuite/gdb.python/py-record-btrace-threads.exp index f0845f8bbd5..ca9552754f3 100644 --- a/gdb/testsuite/gdb.python/py-record-btrace-threads.exp +++ b/gdb/testsuite/gdb.python/py-record-btrace-threads.exp @@ -17,7 +17,7 @@ =20 load_lib gdb-python.exp =20 -require !skip_btrace_tests !skip_python_tests +require allow_btrace_tests !skip_python_tests =20 standard_testfile =20 diff --git a/gdb/testsuite/gdb.python/py-record-btrace.exp b/gdb/testsuite/= gdb.python/py-record-btrace.exp index 1326e8283b9..1b9824e284b 100644 --- a/gdb/testsuite/gdb.python/py-record-btrace.exp +++ b/gdb/testsuite/gdb.python/py-record-btrace.exp @@ -17,7 +17,7 @@ =20 # Skip this test if btrace is disabled. =20 -require !skip_btrace_tests !skip_python_tests +require allow_btrace_tests !skip_python_tests =20 load_lib gdb-python.exp =20 diff --git a/gdb/testsuite/lib/gdb.exp b/gdb/testsuite/lib/gdb.exp index 8d817dfa682..40a12612b3e 100644 --- a/gdb/testsuite/lib/gdb.exp +++ b/gdb/testsuite/lib/gdb.exp @@ -3801,22 +3801,22 @@ gdb_caching_proc allow_avx512fp16_tests { return $allow_avx512fp16_tests } =20 -# Run a test on the target to see if it supports btrace hardware. Return = 0 if so, -# 1 if it does not. Based on 'check_vmx_hw_available' from the GCC testsu= ite. +# Run a test on the target to see if it supports btrace hardware. Return = 1 if so, +# 0 if it does not. Based on 'check_vmx_hw_available' from the GCC testsu= ite. =20 -gdb_caching_proc skip_btrace_tests { +gdb_caching_proc allow_btrace_tests { global srcdir subdir gdb_prompt inferior_exited_re =20 - set me "skip_btrace_tests" + set me "allow_btrace_tests" if { ![istarget "i?86-*-*"] && ![istarget "x86_64-*-*"] } { - verbose "$me: target does not support btrace, returning 1" 2 - return 1 + verbose "$me: target does not support btrace, returning 0" 2 + return 0 } =20 # Compile a test program. set src { int main() { return 0; } } if {![gdb_simple_compile $me $src executable]} { - return 1 + return 0 } =20 # No error message, compilation succeeded so now run it via gdb. @@ -3826,29 +3826,29 @@ gdb_caching_proc skip_btrace_tests { gdb_reinitialize_dir $srcdir/$subdir gdb_load $obj if ![runto_main] { - return 1 + return 0 } # In case of an unexpected output, we return 2 as a fail value. - set skip_btrace_tests 2 + set allow_btrace_tests 2 gdb_test_multiple "record btrace" "check btrace support" { -re "You can't do that when your target is.*\r\n$gdb_prompt $" { - set skip_btrace_tests 1 + set allow_btrace_tests 0 } -re "Target does not support branch tracing.*\r\n$gdb_prompt $" { - set skip_btrace_tests 1 + set allow_btrace_tests 0 } -re "Could not enable branch tracing.*\r\n$gdb_prompt $" { - set skip_btrace_tests 1 + set allow_btrace_tests 0 } -re "^record btrace\r\n$gdb_prompt $" { - set skip_btrace_tests 0 + set allow_btrace_tests 1 } } gdb_exit remote_file build delete $obj =20 - verbose "$me: returning $skip_btrace_tests" 2 - return $skip_btrace_tests + verbose "$me: returning $allow_btrace_tests" 2 + return $allow_btrace_tests } =20 # Run a test on the target to see if it supports btrace pt hardware.