From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from simark.ca by simark.ca with LMTP id iUHfH2qP8mkiaAQAWB0awg (envelope-from ) for ; Wed, 29 Apr 2026 19:08:26 -0400 Authentication-Results: simark.ca; dkim=pass (2048-bit key; unprotected) header.d=linaro.org header.i=@linaro.org header.a=rsa-sha256 header.s=google header.b=kbJ0spUn; dkim-atps=neutral Received: by simark.ca (Postfix, from userid 112) id 7EC601E0B1; Wed, 29 Apr 2026 19:08:26 -0400 (EDT) X-Spam-Checker-Version: SpamAssassin 4.0.1 (2024-03-25) on simark.ca X-Spam-Level: X-Spam-Status: No, score=-2.4 required=5.0 tests=ARC_SIGNED,ARC_VALID,BAYES_00, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI, RCVD_IN_DNSWL_MED,RCVD_IN_MSPIKE_H2,RCVD_IN_VALIDITY_CERTIFIED_BLOCKED, RCVD_IN_VALIDITY_RPBL_BLOCKED,RCVD_IN_VALIDITY_SAFE_BLOCKED autolearn=ham autolearn_force=no version=4.0.1 Received: from vm01.sourceware.org (vm01.sourceware.org [38.145.34.32]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange x25519 server-signature ECDSA (prime256v1) server-digest SHA256) (No client certificate requested) by simark.ca (Postfix) with ESMTPS id B59D91E0BA for ; Wed, 29 Apr 2026 19:08:23 -0400 (EDT) Received: from vm01.sourceware.org (localhost [127.0.0.1]) by sourceware.org (Postfix) with ESMTP id 57CF74BB58FD for ; Wed, 29 Apr 2026 23:08:23 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 57CF74BB58FD Authentication-Results: sourceware.org; dkim=pass (2048-bit key, unprotected) header.d=linaro.org header.i=@linaro.org header.a=rsa-sha256 header.s=google header.b=kbJ0spUn Received: from mail-ua1-x932.google.com (mail-ua1-x932.google.com [IPv6:2607:f8b0:4864:20::932]) by sourceware.org (Postfix) with ESMTPS id 24C4C4358A58 for ; Wed, 29 Apr 2026 23:06:18 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 24C4C4358A58 Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=linaro.org ARC-Filter: OpenARC Filter v1.0.0 sourceware.org 24C4C4358A58 Authentication-Results: server2.sourceware.org; arc=none smtp.remote-ip=2607:f8b0:4864:20::932 ARC-Seal: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1777503978; cv=none; b=RLEi8dHjJl6Ryupu3gWEFwr56OwN57dVN1K9eQt2ckIMRNeDeJjSLESqNrllkd3tu96DhaaEjVkH0qsun2e8EJZn5nr2aLDAD+ntUMrgnNFxx3T1UulCSzjkqXU+UJypwHdEMMEw5sfyvh5p2aIUrFIMQUXPUNLa4s5mn+45K4A= ARC-Message-Signature: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1777503978; c=relaxed/simple; bh=YPq/Z5Nd6Q2i3Br9ZPZFngVGf//x/HN3vIRjp9n0k3Y=; h=DKIM-Signature:From:To:Subject:Date:Message-ID:MIME-Version; b=p6H2uFP++tjiLDwfDhuMp0PKDdTl0xxveLZau7QB7RncCgRd1023pVosPa4u7MuvAk0lZ2aAXDcqnLaXPjhCiAHEQjFNTQmNKJizFXGH+faJF7LFSh1IuAwJh85mm28ZHUQQpgRUhbgTeiG/4GyY1Df5i/UpFP3ZRynmxZqQ+Lw= ARC-Authentication-Results: i=1; server2.sourceware.org DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 24C4C4358A58 Received: by mail-ua1-x932.google.com with SMTP id a1e0cc1a2514c-95699e8e26aso97627241.0 for ; Wed, 29 Apr 2026 16:06:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1777503977; x=1778108777; darn=sourceware.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:to:from:from:to:cc:subject:date:message-id :reply-to; bh=qnGcooOXZl2Jl76ihM695zr+ClwtfasIWqfU1nL7LbM=; b=kbJ0spUnYaVUdJiIn3a8mLYWOExipmcRr2i6D2ydHw5CWtPDfmUYUNl/u5hju9LoG6 lBPyG0D7+cHP+kOQY2RjK7GTjE7v651sNzcORZNw/lMIG/tFZTAAz9H0462iyrnn3NBt JuPAAtGjOavuHTw5row17drLNITZlp44E48+k9iRnJhGn5z296Ke7n8vwFhBsZuJz21i hosMvk0xSYSG5YCth050IajargQQCx/QWS1dpKS4/vrabHbyiX8Mlut8Qvkj9x6C1nkQ 1L8wWpB+u8dIICTxgvQBBKLn69RpsRyBfLxvgZ/wWcLs+DPiHFVsgcWyKedZsGIigMhl LHDA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1777503977; x=1778108777; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:to:from:x-gm-gg:x-gm-message-state:from:to :cc:subject:date:message-id:reply-to; bh=qnGcooOXZl2Jl76ihM695zr+ClwtfasIWqfU1nL7LbM=; b=qSiZVH+rFk0jA1y5YlyxtC8ulhIlX7Eg3BuBH/CS9wZh++k1KAfQ+hmGfMKJAAlB6Z WO/Nb0CgB+9QehgC3lrBJcIzkeFx9a6dLsIrlWjegrgKsWIqu03b9yW+ylJoMs3E10pG 5bz97JrNMNgOVgHZ9WAg5Sol6kjf6EmK/OIVUgFUbgZPN/kvlbMC8aNspXDPPicjUV4s iXSrzH806BmTYPwAcSTYOap9hcjzJK/2I3/BP66IHdPfLC0AAA6BTFETJiRaILTgU9s9 UJ4SoI2M75kaIhVNciLv6TDZk5gTlVjdZos3m2fHKkBPrOoB4fhcVazJmHW8t4D3j/F8 z4kQ== X-Gm-Message-State: AOJu0YzAObqcFlGDINjBjc0U6bJY9Lm+gIYq/r75ZeJdki0fMsA8IY1b ISY/NG7/5fPhU0/qUZRZwnTC66dI+EEcuK08FQFBMhP4CY4LllKF3GB3xYuS2xqvC2wr90sF4zB /tFe6 X-Gm-Gg: AeBDietLZ8rsVwIUrWevMj6upo8iDVvelJ6IVrIUOGjxD4fzJ03QYymzaDd5XdFRpNz PARHz/7IqAeY5jkLmcAjgG1uBpBMO9nMz0rtNi5KZ/jPsKl0T/8qzFz+vKkGIxfvV961A/Cgo1H HmG+0OvZkyFJrtYHfCzdP3YYLCPrfV4VNMev7Gbsmq9fpQYxf2FmsllZwJ0xVwEcdd/82PpBrIf lcWrCW4SnN/ARuSU6At9ZR0KZ6UCINZgaRYzZE4R9KyiYSEKc8Ci8f0t+FC0LO4RkrswR0efMkw LBK+I5vV+DfTfxZ5EC4qqJzjVF8oScUkU/ZB78b4ZWMwamWVkFtSdv0HY7eZDw8Z9cB9UOlYJTn fn3Fnoft1L7IfivDxxqMXwXYJaWO0CFX1IE7n4tTiMuVWUta+8jUBQ6miT5N8uQ66PT9cO5P4QB 0o0T29MZ0QD4NbwVhHK2mjI6loQ9WKnx956cYRM/VgwAmj X-Received: by 2002:a05:6102:8096:b0:605:8143:cb67 with SMTP id ada2fe7eead31-62ad4b89086mr252875137.20.1777503977134; Wed, 29 Apr 2026 16:06:17 -0700 (PDT) Received: from localhost ([2804:14d:7e39:8083:f04c:42e3:5943:38f6]) by smtp.gmail.com with ESMTPSA id ada2fe7eead31-62982136587sm1929800137.2.2026.04.29.16.06.16 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 29 Apr 2026 16:06:16 -0700 (PDT) From: Thiago Jung Bauermann To: gdb-patches@sourceware.org Subject: [PATCH v2 28/37] GDB: testsuite: reverse: Don't return -1 from top-level (sed) Date: Wed, 29 Apr 2026 20:04:19 -0300 Message-ID: <20260429230432.60487-29-thiago.bauermann@linaro.org> X-Mailer: git-send-email 2.54.0 In-Reply-To: <20260429230432.60487-1-thiago.bauermann@linaro.org> References: <20260429230432.60487-1-thiago.bauermann@linaro.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.30 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: gdb-patches-bounces~public-inbox=simark.ca@sourceware.org This patch is purely the result of running: $ sed -i 's/^ return -1/ return/' *.exp* inside gdb/testsuite/gdb.reverse. --- gdb/testsuite/gdb.reverse/aarch64-mops.exp | 4 ++-- gdb/testsuite/gdb.reverse/amd64-tailcall-reverse.exp | 4 ++-- gdb/testsuite/gdb.reverse/basic-record-full.exp | 2 +- gdb/testsuite/gdb.reverse/break-precsave.exp | 2 +- gdb/testsuite/gdb.reverse/break-reverse.exp | 2 +- gdb/testsuite/gdb.reverse/consecutive-precsave.exp | 2 +- gdb/testsuite/gdb.reverse/consecutive-reverse.exp | 2 +- gdb/testsuite/gdb.reverse/finish-precsave.exp | 2 +- gdb/testsuite/gdb.reverse/finish-reverse-bkpt.exp | 2 +- gdb/testsuite/gdb.reverse/finish-reverse-next.exp | 2 +- gdb/testsuite/gdb.reverse/finish-reverse.exp | 2 +- gdb/testsuite/gdb.reverse/fstatat-reverse.exp | 2 +- gdb/testsuite/gdb.reverse/getrandom.exp | 2 +- gdb/testsuite/gdb.reverse/getresuid-reverse.exp | 2 +- gdb/testsuite/gdb.reverse/i386-avx-reverse.exp | 2 +- gdb/testsuite/gdb.reverse/i386-precsave.exp | 2 +- gdb/testsuite/gdb.reverse/i386-reverse.exp | 2 +- gdb/testsuite/gdb.reverse/i386-sse-reverse.exp | 2 +- gdb/testsuite/gdb.reverse/i387-env-reverse.exp | 2 +- gdb/testsuite/gdb.reverse/i387-stack-reverse.exp | 2 +- gdb/testsuite/gdb.reverse/insn-reverse.exp | 2 +- gdb/testsuite/gdb.reverse/machinestate-precsave.exp | 2 +- gdb/testsuite/gdb.reverse/machinestate.exp | 2 +- gdb/testsuite/gdb.reverse/maint-print-instruction.exp | 2 +- gdb/testsuite/gdb.reverse/map-to-same-line.exp | 4 ++-- gdb/testsuite/gdb.reverse/next-reverse-bkpt-over-sr.exp | 2 +- gdb/testsuite/gdb.reverse/pipe-reverse.exp | 2 +- gdb/testsuite/gdb.reverse/ppc_record_test_isa_2_06.exp | 2 +- gdb/testsuite/gdb.reverse/ppc_record_test_isa_3_1.exp | 2 +- gdb/testsuite/gdb.reverse/readv-reverse.exp | 2 +- gdb/testsuite/gdb.reverse/recursion.exp | 2 +- gdb/testsuite/gdb.reverse/recvmsg-reverse.exp | 2 +- gdb/testsuite/gdb.reverse/rerun-prec.exp | 2 +- gdb/testsuite/gdb.reverse/s390-mvcle.exp | 2 +- gdb/testsuite/gdb.reverse/sigall-precsave.exp | 2 +- gdb/testsuite/gdb.reverse/sigall-reverse.exp | 2 +- gdb/testsuite/gdb.reverse/singlejmp-reverse.exp | 2 +- gdb/testsuite/gdb.reverse/solib-precsave.exp | 6 +++--- gdb/testsuite/gdb.reverse/solib-reverse.exp | 6 +++--- gdb/testsuite/gdb.reverse/step-indirect-call-thunk.exp | 4 ++-- gdb/testsuite/gdb.reverse/step-precsave.exp | 2 +- gdb/testsuite/gdb.reverse/step-reverse.exp | 2 +- gdb/testsuite/gdb.reverse/test_ioctl_TCSETSW.exp | 2 +- gdb/testsuite/gdb.reverse/until-precsave.exp | 2 +- gdb/testsuite/gdb.reverse/until-reverse.exp | 2 +- gdb/testsuite/gdb.reverse/waitpid-reverse.exp | 2 +- gdb/testsuite/gdb.reverse/watch-precsave.exp | 2 +- gdb/testsuite/gdb.reverse/watch-reverse.exp | 2 +- 48 files changed, 56 insertions(+), 56 deletions(-) diff --git a/gdb/testsuite/gdb.reverse/aarch64-mops.exp b/gdb/testsuite/gdb.reverse/aarch64-mops.exp index 14602d4feb8a..97f1efe82994 100644 --- a/gdb/testsuite/gdb.reverse/aarch64-mops.exp +++ b/gdb/testsuite/gdb.reverse/aarch64-mops.exp @@ -35,15 +35,15 @@ require allow_aarch64_mops_tests standard_testfile if { [prepare_for_testing "failed to prepare" ${testfile} ${srcfile} \ [list debug additional_flags=-march=armv9.3-a]] } { - return -1 + return } if {![runto_main]} { - return -1 + return } gdb_test_no_output "record full" foreach_with_prefix insn_prefix {"set" "cpy" "cpyf"} { diff --git a/gdb/testsuite/gdb.reverse/amd64-tailcall-reverse.exp b/gdb/testsuite/gdb.reverse/amd64-tailcall-reverse.exp index 6db4b43ac0a1..c3289fa4c6ff 100644 --- a/gdb/testsuite/gdb.reverse/amd64-tailcall-reverse.exp +++ b/gdb/testsuite/gdb.reverse/amd64-tailcall-reverse.exp @@ -25,15 +25,15 @@ if {[info exists COMPILE]} { } else { require is_x86_64_m64_target } if { [prepare_for_testing "failed to prepare" ${testfile} ${srcfile} $opts] } { - return -1 + return } if {![runto_main]} { - return -1 + return } if {[supports_process_record]} { gdb_test_no_output "record" } diff --git a/gdb/testsuite/gdb.reverse/basic-record-full.exp b/gdb/testsuite/gdb.reverse/basic-record-full.exp index 78d6f4d02af6..67655417db28 100644 --- a/gdb/testsuite/gdb.reverse/basic-record-full.exp +++ b/gdb/testsuite/gdb.reverse/basic-record-full.exp @@ -22,11 +22,11 @@ require supports_process_record standard_testfile if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } { - return -1 + return } runto_main gdb_test_no_output "record" "turn on process record" diff --git a/gdb/testsuite/gdb.reverse/break-precsave.exp b/gdb/testsuite/gdb.reverse/break-precsave.exp index bee9fd548f60..994457d4439c 100644 --- a/gdb/testsuite/gdb.reverse/break-precsave.exp +++ b/gdb/testsuite/gdb.reverse/break-precsave.exp @@ -21,11 +21,11 @@ require supports_process_record standard_testfile break-reverse.c set precsave [standard_output_file break.precsave] if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } { - return -1 + return } set foo_location [gdb_get_line_number "break in foo" ] set bar_location [gdb_get_line_number "break in bar" ] set main_location [gdb_get_line_number "break in main"] diff --git a/gdb/testsuite/gdb.reverse/break-reverse.exp b/gdb/testsuite/gdb.reverse/break-reverse.exp index 5aeaf22e6ada..7ba681828d33 100644 --- a/gdb/testsuite/gdb.reverse/break-reverse.exp +++ b/gdb/testsuite/gdb.reverse/break-reverse.exp @@ -19,11 +19,11 @@ require supports_reverse standard_testfile if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } { - return -1 + return } set foo_location [gdb_get_line_number "break in foo" ] set bar_location [gdb_get_line_number "break in bar" ] set main_location [gdb_get_line_number "break in main"] diff --git a/gdb/testsuite/gdb.reverse/consecutive-precsave.exp b/gdb/testsuite/gdb.reverse/consecutive-precsave.exp index 7d57661055e9..9a9132d9a4f1 100644 --- a/gdb/testsuite/gdb.reverse/consecutive-precsave.exp +++ b/gdb/testsuite/gdb.reverse/consecutive-precsave.exp @@ -21,11 +21,11 @@ require supports_process_record standard_testfile consecutive-reverse.c set precsave [standard_output_file consecutive.precsave] if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } { - return -1 + return } runto_main # Activate process record/replay diff --git a/gdb/testsuite/gdb.reverse/consecutive-reverse.exp b/gdb/testsuite/gdb.reverse/consecutive-reverse.exp index c22cb96504b4..1b44a134cb84 100644 --- a/gdb/testsuite/gdb.reverse/consecutive-reverse.exp +++ b/gdb/testsuite/gdb.reverse/consecutive-reverse.exp @@ -19,11 +19,11 @@ require supports_reverse standard_testfile if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } { - return -1 + return } runto_main if {[supports_process_record]} { diff --git a/gdb/testsuite/gdb.reverse/finish-precsave.exp b/gdb/testsuite/gdb.reverse/finish-precsave.exp index 7430b890d545..00c7d8caa419 100644 --- a/gdb/testsuite/gdb.reverse/finish-precsave.exp +++ b/gdb/testsuite/gdb.reverse/finish-precsave.exp @@ -20,11 +20,11 @@ require supports_reverse standard_testfile finish-reverse.c set precsave [standard_output_file finish.precsave] if { [prepare_for_testing "failed to prepare" "$testfile" $srcfile] } { - return -1 + return } runto_main if {[supports_process_record]} { diff --git a/gdb/testsuite/gdb.reverse/finish-reverse-bkpt.exp b/gdb/testsuite/gdb.reverse/finish-reverse-bkpt.exp index 67df198bb051..19ebebf6a126 100644 --- a/gdb/testsuite/gdb.reverse/finish-reverse-bkpt.exp +++ b/gdb/testsuite/gdb.reverse/finish-reverse-bkpt.exp @@ -43,11 +43,11 @@ require supports_reverse standard_testfile if { [prepare_for_testing "failed to prepare" "$testfile" $srcfile] } { - return -1 + return } if {![runto_main]} { return 0 } diff --git a/gdb/testsuite/gdb.reverse/finish-reverse-next.exp b/gdb/testsuite/gdb.reverse/finish-reverse-next.exp index 325282855c64..96af034af1ef 100644 --- a/gdb/testsuite/gdb.reverse/finish-reverse-next.exp +++ b/gdb/testsuite/gdb.reverse/finish-reverse-next.exp @@ -46,11 +46,11 @@ if {![supports_reverse]} { } standard_testfile if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } { - return -1 + return } runto_main if {[supports_process_record]} { diff --git a/gdb/testsuite/gdb.reverse/finish-reverse.exp b/gdb/testsuite/gdb.reverse/finish-reverse.exp index 3f3ef666323a..b234d7fec143 100644 --- a/gdb/testsuite/gdb.reverse/finish-reverse.exp +++ b/gdb/testsuite/gdb.reverse/finish-reverse.exp @@ -19,11 +19,11 @@ require supports_reverse standard_testfile if { [prepare_for_testing "failed to prepare" "$testfile" $srcfile] } { - return -1 + return } runto_main if {[supports_process_record]} { diff --git a/gdb/testsuite/gdb.reverse/fstatat-reverse.exp b/gdb/testsuite/gdb.reverse/fstatat-reverse.exp index 7ccdcd4ae9e8..4c8dfed07b17 100644 --- a/gdb/testsuite/gdb.reverse/fstatat-reverse.exp +++ b/gdb/testsuite/gdb.reverse/fstatat-reverse.exp @@ -22,11 +22,11 @@ require supports_reverse standard_testfile if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } { - return -1 + return } runto_main if {[supports_process_record]} { diff --git a/gdb/testsuite/gdb.reverse/getrandom.exp b/gdb/testsuite/gdb.reverse/getrandom.exp index 410d4ab3bde6..c65de709c4ad 100644 --- a/gdb/testsuite/gdb.reverse/getrandom.exp +++ b/gdb/testsuite/gdb.reverse/getrandom.exp @@ -23,11 +23,11 @@ require supports_reverse require {have_system_header sys/random.h} standard_testfile if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } { - return -1 + return } runto_main if {[supports_process_record]} { diff --git a/gdb/testsuite/gdb.reverse/getresuid-reverse.exp b/gdb/testsuite/gdb.reverse/getresuid-reverse.exp index af8311344bfb..27d10afc4e91 100644 --- a/gdb/testsuite/gdb.reverse/getresuid-reverse.exp +++ b/gdb/testsuite/gdb.reverse/getresuid-reverse.exp @@ -22,11 +22,11 @@ require supports_reverse standard_testfile if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } { - return -1 + return } runto_main if {[supports_process_record]} { diff --git a/gdb/testsuite/gdb.reverse/i386-avx-reverse.exp b/gdb/testsuite/gdb.reverse/i386-avx-reverse.exp index bdb76733e959..2268f2c28722 100644 --- a/gdb/testsuite/gdb.reverse/i386-avx-reverse.exp +++ b/gdb/testsuite/gdb.reverse/i386-avx-reverse.exp @@ -39,11 +39,11 @@ standard_testfile set additional_flags [gdb_target_symbol_prefix_flags] lappend_include_file alloc_lib $srcdir/lib/precise-aligned-alloc.c if {[prepare_for_testing "failed to prepare" $testfile $srcfile \ [list debug $additional_flags $alloc_lib]]} { - return -1 + return } # Shorthand to test reversing through one instruction and # testing if a register has the expected value. # Prefix, if included, should end with a colon and space. diff --git a/gdb/testsuite/gdb.reverse/i386-precsave.exp b/gdb/testsuite/gdb.reverse/i386-precsave.exp index 4841c1b65aa2..4bde399bc457 100644 --- a/gdb/testsuite/gdb.reverse/i386-precsave.exp +++ b/gdb/testsuite/gdb.reverse/i386-precsave.exp @@ -28,11 +28,11 @@ set precsave [standard_output_file i386.precsave] # some targets have leading underscores on assembly symbols. set additional_flags [gdb_target_symbol_prefix_flags] if {[prepare_for_testing "failed to prepare" $testfile $srcfile \ [list debug $additional_flags]]} { - return -1 + return } set end_of_main [gdb_get_line_number " end of main "] set end_of_inc_dec_tests [gdb_get_line_number " end inc_dec_tests "] diff --git a/gdb/testsuite/gdb.reverse/i386-reverse.exp b/gdb/testsuite/gdb.reverse/i386-reverse.exp index 0d27454d0101..2013a969f8ca 100644 --- a/gdb/testsuite/gdb.reverse/i386-reverse.exp +++ b/gdb/testsuite/gdb.reverse/i386-reverse.exp @@ -26,11 +26,11 @@ standard_testfile # some targets have leading underscores on assembly symbols. set additional_flags [gdb_target_symbol_prefix_flags] if {[prepare_for_testing "failed to prepare" $testfile $srcfile \ [list debug $additional_flags]]} { - return -1 + return } set end_of_main [gdb_get_line_number " end of main "] set end_of_inc_dec_tests [gdb_get_line_number " end inc_dec_tests "] diff --git a/gdb/testsuite/gdb.reverse/i386-sse-reverse.exp b/gdb/testsuite/gdb.reverse/i386-sse-reverse.exp index ad8eaeaadc23..dc1308861e2c 100644 --- a/gdb/testsuite/gdb.reverse/i386-sse-reverse.exp +++ b/gdb/testsuite/gdb.reverse/i386-sse-reverse.exp @@ -31,11 +31,11 @@ standard_testfile # some targets have leading underscores on assembly symbols. set additional_flags [gdb_target_symbol_prefix_flags] if {[prepare_for_testing "failed to prepare" $testfile $srcfile \ [list debug $additional_flags]]} { - return -1 + return } set end_of_main [gdb_get_line_number " end of main "] set end_sse_test [gdb_get_line_number " end sse_test "] set end_ssse3_test [gdb_get_line_number " end ssse3_test "] diff --git a/gdb/testsuite/gdb.reverse/i387-env-reverse.exp b/gdb/testsuite/gdb.reverse/i387-env-reverse.exp index 5fc6346f86bf..ebf7d8dec425 100644 --- a/gdb/testsuite/gdb.reverse/i387-env-reverse.exp +++ b/gdb/testsuite/gdb.reverse/i387-env-reverse.exp @@ -23,11 +23,11 @@ standard_testfile # some targets have leading underscores on assembly symbols. set additional_flags [gdb_target_symbol_prefix_flags] if {[prepare_for_testing "failed to prepare" $testfile $srcfile \ [list debug $additional_flags]]} { - return -1 + return } runto_main # Record to end of main diff --git a/gdb/testsuite/gdb.reverse/i387-stack-reverse.exp b/gdb/testsuite/gdb.reverse/i387-stack-reverse.exp index c6d3cca23424..384faade5eee 100644 --- a/gdb/testsuite/gdb.reverse/i387-stack-reverse.exp +++ b/gdb/testsuite/gdb.reverse/i387-stack-reverse.exp @@ -23,11 +23,11 @@ standard_testfile # some targets have leading underscores on assembly symbols. set additional_flags [gdb_target_symbol_prefix_flags] if {[prepare_for_testing "failed to prepare" $testfile $srcfile \ [list debug $additional_flags]]} { - return -1 + return } runto_main # Record to end of main diff --git a/gdb/testsuite/gdb.reverse/insn-reverse.exp b/gdb/testsuite/gdb.reverse/insn-reverse.exp index c92059143ece..5b4f6a28f7af 100644 --- a/gdb/testsuite/gdb.reverse/insn-reverse.exp +++ b/gdb/testsuite/gdb.reverse/insn-reverse.exp @@ -17,11 +17,11 @@ require supports_reverse !use_gdb_stub standard_testfile if {[prepare_for_testing "failed to prepare" $testfile $srcfile \ [list debug]]} { - return -1 + return } if {![runto_main]} { return } diff --git a/gdb/testsuite/gdb.reverse/machinestate-precsave.exp b/gdb/testsuite/gdb.reverse/machinestate-precsave.exp index 74c041143d87..8a11c6d24b98 100644 --- a/gdb/testsuite/gdb.reverse/machinestate-precsave.exp +++ b/gdb/testsuite/gdb.reverse/machinestate-precsave.exp @@ -41,11 +41,11 @@ require supports_process_record standard_testfile machinestate.c ms1.c set precsave [standard_output_file machinestate.precsave] if { [prepare_for_testing "failed to prepare" $testfile \ [list $srcfile $srcfile2]] } { - return -1 + return } set newline "\[\r\n\]+" set beginmain [gdb_get_line_number " begin main " $srcfile] diff --git a/gdb/testsuite/gdb.reverse/machinestate.exp b/gdb/testsuite/gdb.reverse/machinestate.exp index 0026af7704bd..de16fbcbb23a 100644 --- a/gdb/testsuite/gdb.reverse/machinestate.exp +++ b/gdb/testsuite/gdb.reverse/machinestate.exp @@ -39,11 +39,11 @@ require supports_reverse standard_testfile .c ms1.c if { [prepare_for_testing "failed to prepare" $testfile \ [list $srcfile $srcfile2]] } { - return -1 + return } set newline "\[\r\n\]+" set beginmain [gdb_get_line_number " begin main " $srcfile] diff --git a/gdb/testsuite/gdb.reverse/maint-print-instruction.exp b/gdb/testsuite/gdb.reverse/maint-print-instruction.exp index dc3bb6b547af..ad8009b415a9 100644 --- a/gdb/testsuite/gdb.reverse/maint-print-instruction.exp +++ b/gdb/testsuite/gdb.reverse/maint-print-instruction.exp @@ -28,11 +28,11 @@ if {![supports_reverse]} { } standard_testfile if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } { - return -1 + return } proc test_print { has_history level test_name } { gdb_test_multiple "maint print record-instruction $level" $test_name { -re -wrap ".*Not enough recorded history.*" { diff --git a/gdb/testsuite/gdb.reverse/map-to-same-line.exp b/gdb/testsuite/gdb.reverse/map-to-same-line.exp index 4e3c1a50229b..1cfb060dfaab 100644 --- a/gdb/testsuite/gdb.reverse/map-to-same-line.exp +++ b/gdb/testsuite/gdb.reverse/map-to-same-line.exp @@ -31,11 +31,11 @@ require is_c_compiler_gcc require supports_reverse standard_testfile .c .S if { [prepare_for_testing "failed to prepare" ${testfile} ${srcfile}] } { - return -1 + return } set asm_file [standard_output_file $srcfile2] Dwarf::assemble $asm_file { global srcdir subdir srcfile @@ -107,11 +107,11 @@ Dwarf::assemble $asm_file { } } if { [prepare_for_testing "failed to prepare" ${testfile} \ [list $srcfile $asm_file] {nodebug} ] } { - return -1 + return } if { ![runto_main] } { return } diff --git a/gdb/testsuite/gdb.reverse/next-reverse-bkpt-over-sr.exp b/gdb/testsuite/gdb.reverse/next-reverse-bkpt-over-sr.exp index 83162e59dfc6..d34fe130e8e5 100644 --- a/gdb/testsuite/gdb.reverse/next-reverse-bkpt-over-sr.exp +++ b/gdb/testsuite/gdb.reverse/next-reverse-bkpt-over-sr.exp @@ -43,11 +43,11 @@ require supports_reverse standard_testfile if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } { - return -1 + return } if {![runto_main]} { return 0 } diff --git a/gdb/testsuite/gdb.reverse/pipe-reverse.exp b/gdb/testsuite/gdb.reverse/pipe-reverse.exp index d4b99c91a8e7..67035866af9a 100644 --- a/gdb/testsuite/gdb.reverse/pipe-reverse.exp +++ b/gdb/testsuite/gdb.reverse/pipe-reverse.exp @@ -22,11 +22,11 @@ require supports_reverse standard_testfile if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } { - return -1 + return } runto_main if {[supports_process_record]} { diff --git a/gdb/testsuite/gdb.reverse/ppc_record_test_isa_2_06.exp b/gdb/testsuite/gdb.reverse/ppc_record_test_isa_2_06.exp index aa69e5d00c99..6bbe98116385 100644 --- a/gdb/testsuite/gdb.reverse/ppc_record_test_isa_2_06.exp +++ b/gdb/testsuite/gdb.reverse/ppc_record_test_isa_2_06.exp @@ -38,11 +38,11 @@ set executable record_test_isa_2_06 set options [list debug] require {istarget "powerpc*"} if {[build_executable "failed to prepare" $executable $srcfile $options] == -1} { - return -1 + return } clean_restart $executable if {![runto_main]} { diff --git a/gdb/testsuite/gdb.reverse/ppc_record_test_isa_3_1.exp b/gdb/testsuite/gdb.reverse/ppc_record_test_isa_3_1.exp index d4ecbbb7fb66..8435d3d840d6 100644 --- a/gdb/testsuite/gdb.reverse/ppc_record_test_isa_3_1.exp +++ b/gdb/testsuite/gdb.reverse/ppc_record_test_isa_3_1.exp @@ -39,11 +39,11 @@ set executable record_test_isa_3_1 require {istarget "powerpc*"} allow_power_isa_3_1_tests set options [list additional_flags=-mcpu=power10 debug] if {[build_executable "failed to prepare" $executable $srcfile $options] == -1} { - return -1 + return } clean_restart $executable if {![runto_main]} { diff --git a/gdb/testsuite/gdb.reverse/readv-reverse.exp b/gdb/testsuite/gdb.reverse/readv-reverse.exp index e05e3de351b6..e5467d8051f8 100644 --- a/gdb/testsuite/gdb.reverse/readv-reverse.exp +++ b/gdb/testsuite/gdb.reverse/readv-reverse.exp @@ -22,11 +22,11 @@ require supports_reverse standard_testfile if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } { - return -1 + return } runto_main if {[supports_process_record]} { diff --git a/gdb/testsuite/gdb.reverse/recursion.exp b/gdb/testsuite/gdb.reverse/recursion.exp index 6284ebc0c16e..6a81821d62bd 100644 --- a/gdb/testsuite/gdb.reverse/recursion.exp +++ b/gdb/testsuite/gdb.reverse/recursion.exp @@ -19,11 +19,11 @@ require supports_reverse standard_testfile if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } { - return -1 + return } runto_main if {[supports_process_record]} { diff --git a/gdb/testsuite/gdb.reverse/recvmsg-reverse.exp b/gdb/testsuite/gdb.reverse/recvmsg-reverse.exp index ce91b59e7edc..3fa433dde569 100644 --- a/gdb/testsuite/gdb.reverse/recvmsg-reverse.exp +++ b/gdb/testsuite/gdb.reverse/recvmsg-reverse.exp @@ -22,11 +22,11 @@ require supports_reverse standard_testfile if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } { - return -1 + return } runto_main if {[supports_process_record]} { diff --git a/gdb/testsuite/gdb.reverse/rerun-prec.exp b/gdb/testsuite/gdb.reverse/rerun-prec.exp index aed3c9e52266..a1a41b007561 100644 --- a/gdb/testsuite/gdb.reverse/rerun-prec.exp +++ b/gdb/testsuite/gdb.reverse/rerun-prec.exp @@ -16,11 +16,11 @@ require supports_reverse supports_process_record standard_testfile if {[prepare_for_testing "failed to prepare" $testfile [list $srcfile]]} { - return -1 + return } # The bug is a regression in the sequence "run; record; run". runto_main gdb_test_no_output "record" "turn on process record" diff --git a/gdb/testsuite/gdb.reverse/s390-mvcle.exp b/gdb/testsuite/gdb.reverse/s390-mvcle.exp index 4bb89c678b48..12fd622f08db 100644 --- a/gdb/testsuite/gdb.reverse/s390-mvcle.exp +++ b/gdb/testsuite/gdb.reverse/s390-mvcle.exp @@ -20,11 +20,11 @@ require supports_reverse {istarget "s390*-*-*"} standard_testfile if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } { - return -1 + return } runto_main if {[supports_process_record]} { diff --git a/gdb/testsuite/gdb.reverse/sigall-precsave.exp b/gdb/testsuite/gdb.reverse/sigall-precsave.exp index 2635e6fdcda4..d8bc6dfaccfa 100644 --- a/gdb/testsuite/gdb.reverse/sigall-precsave.exp +++ b/gdb/testsuite/gdb.reverse/sigall-precsave.exp @@ -19,11 +19,11 @@ require supports_reverse standard_testfile sigall-reverse.c set precsave [standard_output_file sigall.precsave] if {[build_executable $testfile.exp $testfile $srcfile debug] == -1} { - return -1 + return } proc test_one_sig {nextsig} { global sig_supported global gdb_prompt diff --git a/gdb/testsuite/gdb.reverse/sigall-reverse.exp b/gdb/testsuite/gdb.reverse/sigall-reverse.exp index 5f0ab640d186..2dae1933d818 100644 --- a/gdb/testsuite/gdb.reverse/sigall-reverse.exp +++ b/gdb/testsuite/gdb.reverse/sigall-reverse.exp @@ -18,11 +18,11 @@ require {!target_info exists gdb,nosignals} require supports_reverse standard_testfile if {[build_executable $testfile.exp $testfile $srcfile debug] == -1} { - return -1 + return } proc test_one_sig {nextsig} { global sig_supported global gdb_prompt diff --git a/gdb/testsuite/gdb.reverse/singlejmp-reverse.exp b/gdb/testsuite/gdb.reverse/singlejmp-reverse.exp index 00fd8b8bd957..830292cd6669 100644 --- a/gdb/testsuite/gdb.reverse/singlejmp-reverse.exp +++ b/gdb/testsuite/gdb.reverse/singlejmp-reverse.exp @@ -35,11 +35,11 @@ if {[info exists COMPILE]} { } clean_restart $executable if {![runto_main]} { - return -1 + return } if {[supports_process_record]} { gdb_test_no_output "record" } diff --git a/gdb/testsuite/gdb.reverse/solib-precsave.exp b/gdb/testsuite/gdb.reverse/solib-precsave.exp index 0e0dd3668ac7..ef3229c74927 100644 --- a/gdb/testsuite/gdb.reverse/solib-precsave.exp +++ b/gdb/testsuite/gdb.reverse/solib-precsave.exp @@ -30,16 +30,16 @@ set library2 [standard_output_file ${lib2file}.sl] # Compile the first without debug info so that # stepping and reverse stepping doesn't end up inside them. if { [gdb_compile_shlib ${srcdir}/${subdir}/${lib1src} ${library1} ""] != "" } { untested "failed to compile shared library 1" - return -1 + return } if { [gdb_compile_shlib ${srcdir}/${subdir}/${lib2src} ${library2} "debug"] != "" } { untested "failed to compile shared library 2" - return -1 + return } set exec_opts [list debug shlib=${library1} shlib=${library2}] # Attempt to prevent -Wl,-z,relro which may happen by default with some @@ -48,11 +48,11 @@ set exec_opts [list debug shlib=${library1} shlib=${library2}] if { [gdb_compile ${srcdir}/${subdir}/${srcfile} ${binfile} executable \ [concat $exec_opts ldflags=-Wl,-z,norelro]] != "" && [gdb_compile ${srcdir}/${subdir}/${srcfile} ${binfile} executable $exec_opts] != "" } { untested "failed to compile" - return -1 + return } # Note: The test previously did "set debug-file-directory" to (try to) # ensure the debug info for the dynamic loader and libc weren't found. # This doesn't work if the debug info is in the .debug subdirectory. diff --git a/gdb/testsuite/gdb.reverse/solib-reverse.exp b/gdb/testsuite/gdb.reverse/solib-reverse.exp index 04bbf56757cb..c5a610fcb391 100644 --- a/gdb/testsuite/gdb.reverse/solib-reverse.exp +++ b/gdb/testsuite/gdb.reverse/solib-reverse.exp @@ -28,23 +28,23 @@ set library2 [standard_output_file ${lib2file}.sl] # Compile the first without debug info so that # stepping and reverse stepping doesn't end up inside them. if { [gdb_compile_shlib ${srcdir}/${subdir}/${lib1src} ${library1} ""] != "" } { untested "failed to compile shared library 1" - return -1 + return } if { [gdb_compile_shlib ${srcdir}/${subdir}/${lib2src} ${library2} "debug"] != "" } { untested "failed to compile shared library 2" - return -1 + return } set exec_opts [list debug shlib=${library1} shlib=${library2}] if { [gdb_compile ${srcdir}/${subdir}/${srcfile} ${binfile} executable $exec_opts] != "" } { untested "failed to compile" - return -1 + return } # Note: The test previously did "set debug-file-directory" to (try to) # ensure the debug info for the dynamic loader and libc weren't found. # This doesn't work if the debug info is in the .debug subdirectory. diff --git a/gdb/testsuite/gdb.reverse/step-indirect-call-thunk.exp b/gdb/testsuite/gdb.reverse/step-indirect-call-thunk.exp index 3b00a2fb0895..feb7a840d8a0 100644 --- a/gdb/testsuite/gdb.reverse/step-indirect-call-thunk.exp +++ b/gdb/testsuite/gdb.reverse/step-indirect-call-thunk.exp @@ -29,15 +29,15 @@ if { [gdb_can_simple_compile fcf-protection-none {int main () { return 0; }} \ executable $fcf_protection_none] } { lappend cflags $fcf_protection_none } if { [prepare_for_testing "failed to prepare" $testfile $srcfile $cflags] } { - return -1 + return } if { ![runto_main] } { - return -1 + return } gdb_test_no_output "record" gdb_test "next" ".*" "record trace" diff --git a/gdb/testsuite/gdb.reverse/step-precsave.exp b/gdb/testsuite/gdb.reverse/step-precsave.exp index 33fabd7ec196..761713436771 100644 --- a/gdb/testsuite/gdb.reverse/step-precsave.exp +++ b/gdb/testsuite/gdb.reverse/step-precsave.exp @@ -25,11 +25,11 @@ require supports_process_record standard_testfile step-reverse.c set precsave [standard_output_file step.precsave] if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } { - return -1 + return } runto_main # Activate process record/replay diff --git a/gdb/testsuite/gdb.reverse/step-reverse.exp b/gdb/testsuite/gdb.reverse/step-reverse.exp index b14273e8bcf6..d72638c9169b 100644 --- a/gdb/testsuite/gdb.reverse/step-reverse.exp +++ b/gdb/testsuite/gdb.reverse/step-reverse.exp @@ -23,11 +23,11 @@ require supports_reverse standard_testfile if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } { - return -1 + return } runto_main if {[supports_process_record]} { diff --git a/gdb/testsuite/gdb.reverse/test_ioctl_TCSETSW.exp b/gdb/testsuite/gdb.reverse/test_ioctl_TCSETSW.exp index c3fb0487d187..17d4369ee4ee 100644 --- a/gdb/testsuite/gdb.reverse/test_ioctl_TCSETSW.exp +++ b/gdb/testsuite/gdb.reverse/test_ioctl_TCSETSW.exp @@ -19,11 +19,11 @@ require supports_process_record standard_testfile .c if { [prepare_for_testing "failed to prepare" ${testfile} ${srcfile}] } { - return -1 + return } if {![runto_main]} { untested "could not run to main" continue diff --git a/gdb/testsuite/gdb.reverse/until-precsave.exp b/gdb/testsuite/gdb.reverse/until-precsave.exp index a0bed8dfba51..411fc14fed27 100644 --- a/gdb/testsuite/gdb.reverse/until-precsave.exp +++ b/gdb/testsuite/gdb.reverse/until-precsave.exp @@ -22,11 +22,11 @@ require supports_process_record standard_testfile until-reverse.c ur1.c set precsave [standard_output_file until.precsave] if { [prepare_for_testing "failed to prepare" $testfile \ [list $srcfile $srcfile2]] } { - return -1 + return } set bp_location1 [gdb_get_line_number "set breakpoint 1 here"] set bp_location7 [gdb_get_line_number "set breakpoint 7 here"] set bp_location8 [gdb_get_line_number "set breakpoint 8 here" "$srcfile2"] diff --git a/gdb/testsuite/gdb.reverse/until-reverse.exp b/gdb/testsuite/gdb.reverse/until-reverse.exp index dc8fcbe78d68..74703223c643 100644 --- a/gdb/testsuite/gdb.reverse/until-reverse.exp +++ b/gdb/testsuite/gdb.reverse/until-reverse.exp @@ -20,11 +20,11 @@ require supports_reverse standard_testfile .c ur1.c if { [prepare_for_testing "failed to prepare" $testfile \ [list $srcfile $srcfile2]] } { - return -1 + return } set bp_location1 [gdb_get_line_number "set breakpoint 1 here"] set bp_location7 [gdb_get_line_number "set breakpoint 7 here"] set bp_location8 [gdb_get_line_number "set breakpoint 8 here" "$srcfile2"] diff --git a/gdb/testsuite/gdb.reverse/waitpid-reverse.exp b/gdb/testsuite/gdb.reverse/waitpid-reverse.exp index bc6b32cdc663..dc213259bf0f 100644 --- a/gdb/testsuite/gdb.reverse/waitpid-reverse.exp +++ b/gdb/testsuite/gdb.reverse/waitpid-reverse.exp @@ -24,11 +24,11 @@ require supports_reverse standard_testfile if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } { - return -1 + return } runto_main if {[supports_process_record]} { diff --git a/gdb/testsuite/gdb.reverse/watch-precsave.exp b/gdb/testsuite/gdb.reverse/watch-precsave.exp index 2cc668fdc6a5..474012f74c73 100644 --- a/gdb/testsuite/gdb.reverse/watch-precsave.exp +++ b/gdb/testsuite/gdb.reverse/watch-precsave.exp @@ -22,11 +22,11 @@ require supports_process_record standard_testfile watch-reverse.c set precsave [standard_output_file watch.precsave] if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } { - return -1 + return } runto_main # Activate process record/replay diff --git a/gdb/testsuite/gdb.reverse/watch-reverse.exp b/gdb/testsuite/gdb.reverse/watch-reverse.exp index e165aa084dc7..52b36c6f70ae 100644 --- a/gdb/testsuite/gdb.reverse/watch-reverse.exp +++ b/gdb/testsuite/gdb.reverse/watch-reverse.exp @@ -21,11 +21,11 @@ require supports_reverse standard_testfile if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } { - return -1 + return } runto_main if {[supports_process_record]} {