2004-07-10 Michael Snyder <msnyder@redhat.com>
* symfile.c (generic_load): Comment typo. * stack.c (get_selected_block): Ditto. * regcache.c (regcache_cooked_read): Ditto. * monitor.c (monitor_debug): Ditto. * mips-tdep.c (mips_read_pc): Ditto. * i386-linux-nat.c (ps_get_thread_area): Ditto. * gdb_mbuild.sh: Ditto. * gdbarch.sh: Ditto. * gdbarch.h: Ditto.
This commit is contained in:
parent
109d3db382
commit
b2fa5097bf
@ -1,3 +1,15 @@
|
||||
2004-07-10 Michael Snyder <msnyder@redhat.com>
|
||||
|
||||
* symfile.c (generic_load): Comment typo.
|
||||
* stack.c (get_selected_block): Ditto.
|
||||
* regcache.c (regcache_cooked_read): Ditto.
|
||||
* monitor.c (monitor_debug): Ditto.
|
||||
* mips-tdep.c (mips_read_pc): Ditto.
|
||||
* i386-linux-nat.c (ps_get_thread_area): Ditto.
|
||||
* gdb_mbuild.sh: Ditto.
|
||||
* gdbarch.sh: Ditto.
|
||||
* gdbarch.h: Ditto.
|
||||
|
||||
2004-07-09 Paul N. Hilfinger <Hilfinger@gnat.com>
|
||||
|
||||
Remove ARI problems:
|
||||
|
@ -205,7 +205,7 @@ do
|
||||
trap "exit 1" 1 2 15
|
||||
dir=${builddir}/${target}
|
||||
|
||||
# Should a scratch rebuild be forced, for perhaphs the entire
|
||||
# Should a scratch rebuild be forced, for perhaps the entire
|
||||
# build be skipped?
|
||||
|
||||
if ${force}
|
||||
|
@ -1749,7 +1749,7 @@ extern void set_gdbarch_software_single_step (struct gdbarch *gdbarch, gdbarch_s
|
||||
#endif
|
||||
|
||||
/* FIXME: cagney/2003-08-28: Need to find a better way of selecting the
|
||||
disassembler. Perhaphs objdump can handle it? */
|
||||
disassembler. Perhaps objdump can handle it? */
|
||||
|
||||
typedef int (gdbarch_print_insn_ftype) (bfd_vma vma, struct disassemble_info *info);
|
||||
extern int gdbarch_print_insn (struct gdbarch *gdbarch, bfd_vma vma, struct disassemble_info *info);
|
||||
|
@ -667,7 +667,7 @@ f:=:CORE_ADDR:smash_text_address:CORE_ADDR addr:addr:::core_addr_identity::0
|
||||
# single step. If not, then implement single step using breakpoints.
|
||||
F:=:void:software_single_step:enum target_signal sig, int insert_breakpoints_p:sig, insert_breakpoints_p
|
||||
# FIXME: cagney/2003-08-28: Need to find a better way of selecting the
|
||||
# disassembler. Perhaphs objdump can handle it?
|
||||
# disassembler. Perhaps objdump can handle it?
|
||||
f:TARGET_PRINT_INSN:int:print_insn:bfd_vma vma, struct disassemble_info *info:vma, info:::0:
|
||||
f:=:CORE_ADDR:skip_trampoline_code:CORE_ADDR pc:pc:::generic_skip_trampoline_code::0
|
||||
|
||||
|
@ -738,7 +738,7 @@ ps_get_thread_area (const struct ps_prochandle *ph,
|
||||
|
||||
Is this function needed? I'm guessing that the `base' is the
|
||||
address of a a descriptor that libthread_db uses to find the
|
||||
thread local address base that GDB needs. Perhaphs that
|
||||
thread local address base that GDB needs. Perhaps that
|
||||
descriptor is defined by the ABI. Anyway, given that
|
||||
libthread_db calls this function without prompting (gdb
|
||||
requesting tls base) I guess it needs info in there anyway. */
|
||||
|
@ -825,7 +825,7 @@ pc_is_mips16 (bfd_vma memaddr)
|
||||
return 0;
|
||||
}
|
||||
|
||||
/* MIPS believes that the PC has a sign extended value. Perhaphs the
|
||||
/* MIPS believes that the PC has a sign extended value. Perhaps the
|
||||
all registers should be sign extended for simplicity? */
|
||||
|
||||
static CORE_ADDR
|
||||
|
@ -153,7 +153,7 @@ static void monitor_debug (const char *fmt, ...) ATTR_FORMAT(printf, 1, 2);
|
||||
static int monitor_debug_p = 0;
|
||||
|
||||
/* NOTE: This file alternates between monitor_debug_p and remote_debug
|
||||
when determining if debug information is printed. Perhaphs this
|
||||
when determining if debug information is printed. Perhaps this
|
||||
could be simplified. */
|
||||
|
||||
static void
|
||||
|
@ -845,7 +845,7 @@ regcache_cooked_read (struct regcache *regcache, int regnum, void *buf)
|
||||
else if (regcache->readonly_p
|
||||
&& regnum < regcache->descr->nr_cooked_registers
|
||||
&& regcache->register_valid_p[regnum])
|
||||
/* Read-only register cache, perhaphs the cooked value was cached? */
|
||||
/* Read-only register cache, perhaps the cooked value was cached? */
|
||||
memcpy (buf, register_buffer (regcache, regnum),
|
||||
regcache->descr->sizeof_register[regnum]);
|
||||
else
|
||||
|
@ -1578,7 +1578,7 @@ get_selected_block (CORE_ADDR *addr_in_block)
|
||||
return 0;
|
||||
|
||||
/* NOTE: cagney/2002-11-28: Why go to all this effort to not create
|
||||
a selected/current frame? Perhaphs this function is called,
|
||||
a selected/current frame? Perhaps this function is called,
|
||||
indirectly, by WFI in "infrun.c" where avoiding the creation of
|
||||
an inner most frame is very important (it slows down single
|
||||
step). I suspect, though that this was true in the deep dark
|
||||
|
@ -1532,7 +1532,7 @@ generic_load (char *args, int from_tty)
|
||||
to a comment from remote-mips.c (where a call to symbol_file_add
|
||||
was commented out), making the call confuses GDB if more than one
|
||||
file is loaded in. Some targets do (e.g., remote-vx.c) but
|
||||
others don't (or didn't - perhaphs they have all been deleted). */
|
||||
others don't (or didn't - perhaps they have all been deleted). */
|
||||
|
||||
print_transfer_performance (gdb_stdout, cbdata.data_count,
|
||||
cbdata.write_count, end_time - start_time);
|
||||
|
Loading…
Reference in New Issue
Block a user