RE: [PATCH][gdb/testsuite] Fix gdb.btrace/tsx.exp on system with tsx disabled in microcode

["Metzger, Markus T" <markus.t.metzger@intel.com>]

Hello Tom,


>diff --git a/gdb/testsuite/gdb.btrace/tsx.exp b/gdb/testsuite/gdb.btrace/tsx.exp
>index ccde1ea807e..7f96313f1b1 100644
>--- a/gdb/testsuite/gdb.btrace/tsx.exp
>+++ b/gdb/testsuite/gdb.btrace/tsx.exp
>@@ -59,6 +59,11 @@ set abort_2 [multi_line \
>     "\[0-9\]*\t   0x\[0-9a-f\]+ <test\\+\[0-9\]+>:\tmov\[^\\\r\\\n\]*" \
>     "\[0-9\]*\t   0x\[0-9a-f\]+ <test\\+\[0-9\]+>:\tret\[^\\\r\\\n\]*" \
>     ]
>+set abort_3 \
>+    [multi_line \
>+	 "$decimal\t   $hex <main\\+$decimal>:\tcall\[^\\\r\\\n\]*" \
>+	 "$decimal\t   $hex <test\\+$decimal>:\tmov\[^\\\r\\\n\]*" \
>+	 "$decimal\t   $hex <test\\+$decimal>:\tret\[^\\\r\\\n\]*"]

The patterns do not include the call since this is compiler-generated.  The actual
TSX test is written in assembly so we know the instructions.


> set test "speculation indication"
> gdb_test_multiple "record instruction-history" $test {
>@@ -68,6 +73,9 @@ gdb_test_multiple "record instruction-history" $test {
>     -re "$abort_2.*$gdb_prompt $" {
>         pass $test
>     }
>+    -re -wrap "$abort_3" {
>+        pass $gdb_test_name
>+    }

Does this '-wrap' add ".*$gdb_prompt $"?

Note that we need the ".*" after the pattern since this code is compiler-generated
and we don't really know when we will stop after returning from test ().

Regards,
Markus.

Intel Deutschland GmbH
Registered Address: Am Campeon 10, 85579 Neubiberg, Germany
Tel: +49 89 99 8853-0, www.intel.de <http://www.intel.de>
Managing Directors: Christin Eisenschmid, Sharon Heck, Tiffany Doon Silva  
Chairperson of the Supervisory Board: Nicole Lau
Registered Office: Munich
Commercial Register: Amtsgericht Muenchen HRB 186928