OpenE2K
/
binutils-gdb
12
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Don't rely on ecs->wait_for_more. 

* infrun.c (proceed): Clear the stepping state, set
	previous_inferior_ptid and clear infwait state.
	(wait_for_inferior): Don't clear the stepping state, set
	previous_inferior_ptid, or clear the infwait state here.
	(fetch_inferior_event): Don't clear the stepping state, set
	previous_inferior_ptid, or clear the infwait state here.  Don't
	condition on wait_for_more.
This commit is contained in:
Pedro Alves 2008-07-09 22:36:51 +00:00
parent 0d1e5fa71a
commit 59f0d5d953
2 changed files with 28 additions and 27 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
gdb/ChangeLog
View File

@ -1,3 +1,15 @@
2008-07-09 Pedro Alves <pedro@codesourcery.com>
Don't rely on ecs->wait_for_more.
* infrun.c (proceed): Clear the stepping state, set
previous_inferior_ptid and clear infwait state.
(wait_for_inferior): Don't clear the stepping state, set
previous_inferior_ptid, or clear the infwait state here.
(fetch_inferior_event): Don't clear the stepping state, set
previous_inferior_ptid, or clear the infwait state here. Don't
condition on wait_for_more.
2008-07-09 Pedro Alves <pedro@codesourcery.com>
Refactor infrun a bit.

43
gdb/infrun.c
View File

@ -1288,6 +1288,15 @@ proceed (CORE_ADDR addr, enum target_signal siggnal, int step)
updated correctly when the inferior is stopped. */
prev_pc = regcache_read_pc (get_current_regcache ());
/* Fill in with reasonable starting values. */
init_thread_stepping_state (tss);
/* We'll update this if & when we switch to a new thread. */
previous_inferior_ptid = inferior_ptid;
/* Reset to normal state. */
init_infwait_state ();
/* Resume inferior. */
resume (oneproc || step || bpstat_should_step (), stop_signal);
@ -1460,15 +1469,6 @@ wait_for_inferior (int treat_exec_as_sigtrap)
ecs = &ecss;
memset (ecs, 0, sizeof (*ecs));
/* Fill in with reasonable starting values. */
init_thread_stepping_state (tss);
/* Reset to normal state. */
init_infwait_state ();
/* We'll update this if & when we switch to a new thread. */
previous_inferior_ptid = inferior_ptid;
overlay_cache_invalid = 1;
/* We have to invalidate the registers BEFORE calling target_wait
@ -1519,26 +1519,15 @@ fetch_inferior_event (void *client_data)
memset (ecs, 0, sizeof (*ecs));
if (!ecs->wait_some_more)
{
/* Fill in with reasonable starting values. */
init_thread_stepping_state (tcs);
overlay_cache_invalid = 1;
init_infwait_state ();
/* We have to invalidate the registers BEFORE calling target_wait
because they can be loaded from the target while in target_wait.
This makes remote debugging a bit more efficient for those
targets that provide critical registers as part of their normal
status mechanism. */
/* We'll update this if & when we switch to a new thread. */
previous_inferior_ptid = inferior_ptid;
overlay_cache_invalid = 1;
/* We have to invalidate the registers BEFORE calling target_wait
because they can be loaded from the target while in target_wait.
This makes remote debugging a bit more efficient for those
targets that provide critical registers as part of their normal
status mechanism. */
registers_changed ();
}
registers_changed ();
if (deprecated_target_wait_hook)
ecs->ptid =