Files
gdb/gdb-testsuite-make-gdb.reverse-time-reverse.exp-more.patch
Tom de Vries 0e155d6874 - Simplify outdated comment in gdb.spec.
- Update to fedora rawhide @ 0481d32.
  Maintenance script import-fedora.sh:
  * Skip gdb-fix-bg-execution-repeat.patch.
- Add MIT in License tag due to gdbsupport/unordered_dense.h.
- Update to fedora rawhide @ 89e3933.
  Maintenance script import-fedora.sh:
  * Drop gdb-6.8-bz466901-backtrace-full-prelinked.patch from
    skip_patches.
- Update to fedora rawhide @ 660c52f.
  Drop patches:
  * gdb-rhbz1149205-catch-syscall-after-fork-test.patch
- Update to fedora rawhide @ 885cdf8.
  Update patches:
  * gdb-add-rpm-suggestion-script.patch
  Drop patches:
  * fixup-gdb-add-rpm-suggestion-script.patch
- Update to fedora rawhide @ 9c718a5.
  Drop patches:
  * gdb-6.3-mapping-zero-inode-test.patch
- Update to fedora rawhide @ 15778f3.
  Drop patches:
  * gdb-archer-next-over-throw-cxx-exec.patch
- Update to fedora rawhide @ 152468c.
  Maintenance script import-fedora.sh:
  * Rename gdb-6.3-rh-testversion-20041202.patch to
    gdb-test-show-version.patch in skip_patches.
  Patches dropped (Renamed to ...):
  * gdb-add-rpm-suggestion-script.patch
  Patches added (... this):
  * gdb-rpm-suggestion-script.patch
- Move some patches from "Backport from gdb-patches" to
  "Backports from master, available in GDB 17".
- Add patches (swo#33000):
  * gdb-tdep-fix-gdb.ada-finish-var-size.exp-on-ppc64le-.patch
- Add patches (swo#32409):
  * gdb-ada-fix-gdb.ada-overloads.exp-on-s390x.patch
- Add patches:
  * gdb-testsuite-make-gdb.reverse-time-reverse.exp-more.patch
  * gdb-testsuite-fix-gdb.reverse-time-reverse.exp-timeo.patch
  * gdb-testsuite-handle-asm-frame-in-gdb.python-py-miss.patch
  * gdb-testsuite-fix-gdb.base-ptype.exp-with-gcc-15.patch
  * gdb-testsuite-fix-gdb.python-py-objfile.exp-with-gcc.patch
  * gdb-testsuite-fix-gdb.ada-scalar_storage.exp-on-s390.patch
  * gdb-testsuite-fix-gdb.base-bp-permanent.exp-with-gcc.patch
  * gdb-testsuite-don-t-run-to-main-in-gdb.cp-cplusfuncs.patch

OBS-URL: https://build.opensuse.org/package/show/devel:gcc/gdb?expand=0&rev=465
2025-06-05 13:03:19 +00:00

90 lines
2.9 KiB
Diff

From 5a533536994a8855e54e593ec83816e366ee35c9 Mon Sep 17 00:00:00 2001
From: Tom de Vries <tdevries@suse.de>
Date: Fri, 2 May 2025 16:48:14 +0200
Subject: [PATCH 2/6] [gdb/testsuite] Make gdb.reverse/time-reverse.exp more
robust
I noticed that test-case gdb.reverse/time-reverse.exp contains:
...
if [supports_process_record] {
# Activate process record/replay
gdb_test_no_output "record" "turn on process record"
...
So I tried out forcing supports_process_record to 0, and got:
...
FAIL: gdb.reverse/time-reverse.exp: mode=syscall: info record
FAIL: gdb.reverse/time-reverse.exp: mode=syscall: reverse to marker1
FAIL: gdb.reverse/time-reverse.exp: mode=syscall: check time record
FAIL: gdb.reverse/time-reverse.exp: mode=c: info record
FAIL: gdb.reverse/time-reverse.exp: mode=c: reverse to marker1
FAIL: gdb.reverse/time-reverse.exp: mode=c: check time record
...
Fix this by requiring supports_process_record alongside supports_reverse.
I also noticed when running make-check-all.sh that there were a lot of failures
with target board dwarf5-fission-debug-types.
Fix this by not ignoring the result of "runto marker1".
Then I noticed that $srcfile is used as a regexp. Fix this by applying
string_to_regexp.
Tested on x86_64-linux.
Approved-By: Guinevere Larsen <guinevere@redhat.com>
---
gdb/testsuite/gdb.reverse/time-reverse.exp | 19 +++++++++++--------
1 file changed, 11 insertions(+), 8 deletions(-)
diff --git a/gdb/testsuite/gdb.reverse/time-reverse.exp b/gdb/testsuite/gdb.reverse/time-reverse.exp
index 35415a0fe1f..98085f0ebc9 100644
--- a/gdb/testsuite/gdb.reverse/time-reverse.exp
+++ b/gdb/testsuite/gdb.reverse/time-reverse.exp
@@ -20,6 +20,7 @@
#
require supports_reverse
+require supports_process_record
standard_testfile
@@ -38,23 +39,25 @@ proc test {mode} {
return
}
- runto marker1
-
- if [supports_process_record] {
- # Activate process record/replay
- gdb_test_no_output "record" "turn on process record"
+ if { ![runto marker1] } {
+ return
}
+ # Activate process record/replay
+ gdb_test_no_output "record" "turn on process record"
+
+ set re_srcfile [string_to_regexp $::srcfile]
+
gdb_test "break marker2" \
- "Breakpoint $::decimal at $::hex: file .*$::srcfile, line $::decimal.*" \
+ "Breakpoint $::decimal at $::hex: file .*$re_srcfile, line $::decimal.*" \
"set breakpoint at marker2"
- gdb_continue_to_breakpoint "marker2" ".*$::srcfile:.*"
+ gdb_continue_to_breakpoint "marker2" ".*$re_srcfile:.*"
# Show how many instructions we've recorded.
gdb_test "info record" "Active record target: .*"
- gdb_test "reverse-continue" ".*$::srcfile:$::decimal.*" "reverse to marker1"
+ gdb_test "reverse-continue" ".*$re_srcfile:$::decimal.*" "reverse to marker1"
# If the variable was recorded properly, the old contents (-1)
# will be remembered. If not, new contents (current time) will be
--
2.43.0