testsuite, gdb.btrace: remove assembly-check in delta.exp
authorMarkus Metzger <markus.t.metzger@intel.com>
Tue, 15 Dec 2020 11:46:58 +0000 (12:46 +0100)
committerMarkus Metzger <markus.t.metzger@intel.com>
Wed, 3 Mar 2021 08:13:32 +0000 (09:13 +0100)
In gdb.btrace/delta.exp, we test that we do not extend the trace
unintentionally.  This can be tested by checking the number of
instructions.

If we wanted to check the instruction history, as well, we'd need to work
on an assembly file to have deterministic behaviour.  This isn't really
necessary for this test, however, and covered elsewhere.  Also remove the
function call history check for the same reason.

gdb/testsuite/ChangeLog
gdb/testsuite/gdb.btrace/delta.exp

index 9bc6995aa87ac29aa8a153ea1d5b5ce73a42a4a0..1f0941e13e1840d96c044745bcba29422616b327 100644 (file)
@@ -1,3 +1,8 @@
+2021-03-03  Markus Metzger  <markus.t.metzger@intel.com>
+
+       * gdb.btrace/delta.exp: Remove instruction-history and
+       function-call-history checks.
+
 2021-03-03  Markus Metzger  <markus.t.metzger@intel.com>
 
        * README (Note): Add nopie_ldflag.
index 9f6d741c01366dc8641cc6f39d74dd45e0da7521..0d61e9aa54ed339866e0ef015099f3f95c2e5925 100644 (file)
@@ -58,9 +58,6 @@ proc check_trace {} {
     "Recording format: .*" \
     "Recorded 1 instructions in 1 functions \\\(0 gaps\\\) for .*" \
     ]
-  gdb_test "record instruction-history /f 1" \
-    "1\t   0x\[0-9a-f\]+ <\\+\[0-9\]+>:\tmov *\\\$0x0,%eax\r"
-  gdb_test "record function-call-history /c 1" "1\tmain"
 }
 
 # make sure we don't extend the trace when we ask twice.
@@ -86,9 +83,5 @@ with_test_prefix "reverse" {
 # and back
 with_test_prefix "forward" {
     gdb_test "stepi"
-    gdb_test "info record" [multi_line \
-      "Active record target: record-btrace" \
-      "Recording format: .*" \
-      "Recorded 1 instructions in 1 functions \\\(0 gaps\\\) for .*" \
-    ]
+    check_trace
 }