+2020-02-03 Simon Marchi <simon.marchi@efficios.com>
+
+ * fork-child.c (gdb_startup_inferior): Use bool instead of int.
+ * gdbthread.h (class thread_info) <resumed>: Likewise.
+ * infrun.c (resume_1): Likewise.
+ (proceed): Likewise.
+ (infrun_thread_stop_requested): Likewise.
+ (stop_all_threads): Likewise.
+ (handle_inferior_event): Likewise.
+ (restart_threads): Likewise.
+ (finish_step_over): Likewise.
+ (keep_going_stepped_thread): Likewise.
+ * linux-nat.c (attach_proc_task_lwp_callback): Likewise.
+ (linux_handle_extended_wait): Likewise.
+ * record-btrace.c (get_thread_current_frame_id): Likewise.
+ * record-full.c (record_full_wait_1): Likewise.
+ * remote.c (remote_target::process_initial_stop_replies): Likewise.
+ * target.c (target_resume): Likewise.
+ * thread.c (set_running_thread): Likewise.
+
2020-02-03 Alok Kumar Sharma <AlokKumar.Sharma@amd.com>
* f-valprint.c (f77_print_array_1): Changed datatype of index
ptid_t ptid = startup_inferior (proc_target, pid, num_traps, NULL, NULL);
/* Mark all threads non-executing. */
- set_executing (proc_target, ptid, 0);
+ set_executing (proc_target, ptid, false);
return ptid;
}
if the thread does not have a user-given name. */
char *name = NULL;
- /* Non-zero means the thread is executing. Note: this is different
+ /* True means the thread is executing. Note: this is different
from saying that there is an active target and we are stopped at
a breakpoint, for instance. This is a real indicator whether the
thread is off and running. */
bool executing = false;
- /* Non-zero if this thread is resumed from infrun's perspective.
+ /* True if this thread is resumed from infrun's perspective.
Note that a thread can be marked both as not-executing and
resumed at the same time. This happens if we try to resume a
thread that has a wait status pending. We shouldn't let the
thread really run until that wait status has been processed, but
we should not process that wait status if we didn't try to let
the thread run. */
- int resumed = 0;
+ bool resumed = false;
/* Frontend view of the thread state. Note that the THREAD_RUNNING/
THREAD_STOPPED states are different from EXECUTING. When the
}
tp->inf->process_target ()->threads_executing = true;
- tp->resumed = 1;
+ tp->resumed = true;
/* FIXME: What should we do if we are supposed to resume this
thread with a signal? Maybe we should maintain a queue of
resume_ptid = internal_resume_ptid (user_step);
do_target_resume (resume_ptid, 0, GDB_SIGNAL_0);
- tp->resumed = 1;
+ tp->resumed = true;
return;
}
}
}
do_target_resume (resume_ptid, step, sig);
- tp->resumed = 1;
+ tp->resumed = true;
}
/* Resume the inferior. SIG is the signal to give the inferior
inferior function, as in that case we pretend the inferior
doesn't run at all. */
if (!cur_thr->control.in_infcall)
- set_running (resume_target, resume_ptid, 1);
+ set_running (resume_target, resume_ptid, true);
if (debug_infrun)
fprintf_unfiltered (gdb_stdlog,
/* Otherwise we can process the (new) pending event now. Set
it so this pending event is considered by
do_target_wait. */
- tp->resumed = 1;
+ tp->resumed = true;
}
}
/* The thread may be not executing, but still be
resumed with a pending status to process. */
- t->resumed = 0;
+ t->resumed = false;
}
}
t->stop_requested = 0;
t->executing = 0;
- t->resumed = 0;
+ t->resumed = false;
t->control.may_range_step = 0;
/* This may be the first time we see the inferior report
else
mark_ptid = ecs->ptid;
- set_executing (ecs->target, mark_ptid, 0);
+ set_executing (ecs->target, mark_ptid, false);
/* Likewise the resumed flag. */
- set_resumed (ecs->target, mark_ptid, 0);
+ set_resumed (ecs->target, mark_ptid, false);
}
switch (ecs->ws.kind)
"infrun: restart threads: "
"[%s] has pending status\n",
target_pid_to_str (tp->ptid).c_str ());
- tp->resumed = 1;
+ tp->resumed = true;
continue;
}
/* This was cleared early, by handle_inferior_event. Set it
so this pending event is considered by
do_target_wait. */
- tp->resumed = 1;
+ tp->resumed = true;
gdb_assert (!tp->executing);
get_frame_address_space (frame),
tp->suspend.stop_pc);
- tp->resumed = 1;
+ tp->resumed = true;
resume_ptid = internal_resume_ptid (tp->control.stepping_command);
do_target_resume (resume_ptid, 0, GDB_SIGNAL_0);
}
matching libthread_db is not found (or the process uses
raw clone). */
add_thread (linux_target, lp->ptid);
- set_running (linux_target, lp->ptid, 1);
- set_executing (linux_target, lp->ptid, 1);
+ set_running (linux_target, lp->ptid, true);
+ set_executing (linux_target, lp->ptid, true);
}
return 1;
internal to this module, from the perspective of infrun
and the user/frontend, this new thread is running until
it next reports a stop. */
- set_running (linux_target, new_lp->ptid, 1);
- set_executing (linux_target, new_lp->ptid, 1);
+ set_running (linux_target, new_lp->ptid, true);
+ set_executing (linux_target, new_lp->ptid, true);
if (WSTOPSIG (status) != SIGSTOP)
{
get_thread_current_frame_id (struct thread_info *tp)
{
struct frame_id id;
- int executing;
+ bool executing;
/* Set current thread, which is implicitly used by
get_current_frame. */
/* Try to insert the software single step breakpoint.
If insert success, set step to 0. */
- set_executing (proc_target, inferior_ptid, 0);
+ set_executing (proc_target, inferior_ptid, false);
reinit_frame_cache ();
step = !insert_single_step_breakpoints (gdbarch);
- set_executing (proc_target, inferior_ptid, 1);
+ set_executing (proc_target, inferior_ptid, true);
}
if (record_debug)
|| ws.value.sig != GDB_SIGNAL_0)
evthread->suspend.waitstatus_pending_p = 1;
- set_executing (this, event_ptid, 0);
- set_running (this, event_ptid, 0);
+ set_executing (this, event_ptid, false);
+ set_running (this, event_ptid, false);
get_remote_thread_info (evthread)->vcont_resumed = 0;
}
/* We only set the internal executing state here. The user/frontend
running state is set at a higher level. This also clears the
thread's stop_pc as side effect. */
- set_executing (curr_target, ptid, 1);
+ set_executing (curr_target, ptid, true);
clear_inline_frame_state (curr_target, ptid);
}
/* Helper for set_running, that marks one thread either running or
stopped. */
-static int
-set_running_thread (struct thread_info *tp, int running)
+static bool
+set_running_thread (struct thread_info *tp, bool running)
{
- int started = 0;
+ bool started = false;
if (running && tp->state == THREAD_STOPPED)
- started = 1;
+ started = true;
tp->state = running ? THREAD_RUNNING : THREAD_STOPPED;
if (!running)