diff options
| author | Tom Tromey <tromey@redhat.com> | 2014-07-01 11:43:00 -0600 |
|---|---|---|
| committer | Tom Tromey <tromey@redhat.com> | 2014-07-14 08:33:05 -0600 |
| commit | 548740d6bdd115da2c9c17b194016c2c4c0a4c69 (patch) | |
| tree | c6d07c4fc8f734c515bdd150d9b6e68d95c6c218 /gdb/target.c | |
| parent | 0ed82684efcb5243b66f918552403ff4ec1d7ee1 (diff) | |
| download | gdb-548740d6bdd115da2c9c17b194016c2c4c0a4c69.zip gdb-548740d6bdd115da2c9c17b194016c2c4c0a4c69.tar.gz gdb-548740d6bdd115da2c9c17b194016c2c4c0a4c69.tar.bz2 | |
fix record "run" regression
This fixes the record "run" regression pointed out by Marc Khouzam:
https://sourceware.org/ml/gdb/2014-06/msg00096.html
The bug is that target_require_runnable must agree with the handling
of the "run" target, but currently it is out of sync. This patch
fixes the problem by changing target_require_runnable to also ignore
the record_stratum.
Built and regtested on x86-64 Fedora 20.
New test case included.
2014-07-14 Tom Tromey <tromey@redhat.com>
* target.c (target_require_runnable): Also check record_stratum.
Update comment.
2014-07-14 Tom Tromey <tromey@redhat.com>
* gdb.reverse/rerun-prec.c: New file.
* gdb.reverse/rerun-prec.exp: New file.
Diffstat (limited to 'gdb/target.c')
| -rw-r--r-- | gdb/target.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/gdb/target.c b/gdb/target.c index c9c5e4b..07d029a 100644 --- a/gdb/target.c +++ b/gdb/target.c @@ -2425,10 +2425,11 @@ target_require_runnable (void) if (t->to_create_inferior != NULL) return; - /* Do not worry about thread_stratum targets that can not + /* Do not worry about targets at certain strata that can not create inferiors. Assume they will be pushed again if necessary, and continue to the process_stratum. */ if (t->to_stratum == thread_stratum + || t->to_stratum == record_stratum || t->to_stratum == arch_stratum) continue; |