diff options
| author | Yao Qi <yao.qi@linaro.org> | 2015-12-02 09:32:34 +0000 |
|---|---|---|
| committer | Yao Qi <yao.qi@linaro.org> | 2015-12-02 09:32:34 +0000 |
| commit | 87a8c81ef09b8f86844924c71b2d886b58991c60 (patch) | |
| tree | d2fdf6479eab6ba2342d4c48a4a90d7590911734 | |
| parent | 01382ffd00d7072cb70d34c26c0de144d1569932 (diff) | |
| download | gdb-87a8c81ef09b8f86844924c71b2d886b58991c60.zip gdb-87a8c81ef09b8f86844924c71b2d886b58991c60.tar.gz gdb-87a8c81ef09b8f86844924c71b2d886b58991c60.tar.bz2 | |
Fix regression by Do not skip prologue for asm (.S) files
Patch "Do not skip prologue for asm (.S) files" [1] changes GDB's
behaviour on which test gdb.arch/thumb-singlestep.exp depends, so
it causes the fail below:
(gdb) si^M
37 blx foo^M
(gdb) FAIL: gdb.arch/thumb-singlestep.exp: step into foo
the test assumes the program will stop at the instruction after "push"
but it doesn't. The fix to this fail is to do one more single step.
[1] https://sourceware.org/ml/gdb-patches/2015-06/msg00561.html
gdb/testsuite:
2015-12-02 Yao Qi <yao.qi@linaro.org>
* gdb.arch/thumb-singlestep.exp: Do one more single step.
| -rw-r--r-- | gdb/testsuite/ChangeLog | 4 | ||||
| -rw-r--r-- | gdb/testsuite/gdb.arch/thumb-singlestep.exp | 3 |
2 files changed, 7 insertions, 0 deletions
diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog index 12754fc..dd253e3 100644 --- a/gdb/testsuite/ChangeLog +++ b/gdb/testsuite/ChangeLog @@ -1,3 +1,7 @@ +2015-12-02 Yao Qi <yao.qi@linaro.org> + + * gdb.arch/thumb-singlestep.exp: Do one more single step. + 2015-08-21 Gary Benson <gbenson@redhat.com> * gdb.trace/pending.exp: Cope with remote transfer warnings. diff --git a/gdb/testsuite/gdb.arch/thumb-singlestep.exp b/gdb/testsuite/gdb.arch/thumb-singlestep.exp index 07adaaf..e7a865f 100644 --- a/gdb/testsuite/gdb.arch/thumb-singlestep.exp +++ b/gdb/testsuite/gdb.arch/thumb-singlestep.exp @@ -34,5 +34,8 @@ if ![runto_main] then { return -1 } +# GDB doesn't skip prologue for asm files, so do one single step to +# pass instruction "push". +gdb_test "si" "blx foo.*" "single step" gdb_test "si" "foo \\(\\) at .*${srcfile}.*mov r0,#42.*" "step into foo" |