gdb.trace: Fix unavailable.exp if last register happens to be PC.
Commit Message
unavailable.exp executes "info registers", expecting to find at least
two instances of "<unavailable>". However, it uses
"<unavailable>.*<unavailable>" as the pattern, which doesn't match
when the last register happens to be available (eg. PC). Change it
to ".*<unavailable>.*<unavailable>.*" instead.
Noticed on s390, no regression on x86_64.
gdb/testsuite/ChangeLog:
* gdb.trace/unavailable.exp (gdb_unavailable_registers_test_1): Fix
info registers pattern.
---
gdb/testsuite/ChangeLog | 5 +++++
gdb/testsuite/gdb.trace/unavailable.exp | 2 +-
2 files changed, 6 insertions(+), 1 deletion(-)
Comments
Marcin Kościelnicki <koriakin@0x04.net> writes:
> unavailable.exp executes "info registers", expecting to find at least
> two instances of "<unavailable>". However, it uses
> "<unavailable>.*<unavailable>" as the pattern, which doesn't match
> when the last register happens to be available (eg. PC). Change it
> to ".*<unavailable>.*<unavailable>.*" instead.
>
> Noticed on s390, no regression on x86_64.
>
> gdb/testsuite/ChangeLog:
>
> * gdb.trace/unavailable.exp (gdb_unavailable_registers_test_1): Fix
> info registers pattern.
Patch looks good to me.
On 22/02/16 18:37, Yao Qi wrote:
> Marcin Kościelnicki <koriakin@0x04.net> writes:
>
>> unavailable.exp executes "info registers", expecting to find at least
>> two instances of "<unavailable>". However, it uses
>> "<unavailable>.*<unavailable>" as the pattern, which doesn't match
>> when the last register happens to be available (eg. PC). Change it
>> to ".*<unavailable>.*<unavailable>.*" instead.
>>
>> Noticed on s390, no regression on x86_64.
>>
>> gdb/testsuite/ChangeLog:
>>
>> * gdb.trace/unavailable.exp (gdb_unavailable_registers_test_1): Fix
>> info registers pattern.
>
> Patch looks good to me.
>
Thanks, pushed.
@@ -1,3 +1,8 @@
+2016-02-22 Marcin Kościelnicki <koriakin@0x04.net>
+
+ * gdb.trace/unavailable.exp (gdb_unavailable_registers_test_1): Fix
+ info registers pattern.
+
2016-02-18 Iain Buclaw <ibuclaw@gdcproject.org>
* lib/future.exp: Add D support.
@@ -330,7 +330,7 @@ proc gdb_unavailable_registers_test_1 { } {
test_register "\$pc"
gdb_test "info registers" \
- "<unavailable>.*<unavailable>" \
+ ".*<unavailable>.*<unavailable>.*" \
"info registers, multiple registers not available"
gdb_test "info registers \$$spreg" \