* symfile.c (reread_symbols): Include bfd_errmsg string in error
[deliverable/binutils-gdb.git] / gdb / ChangeLog
index b309561ba615100d3ab04d4882ec03f109075165..9ec1604e7e6d7e442fa4bde6a8a2d3336a90c5a2 100644 (file)
@@ -1,3 +1,345 @@
+Fri Oct 14 10:29:08 1994  Jim Kingdon  (kingdon@lioth.cygnus.com)
+
+       * exec.c (map_vmap): Cast return from xmalloc to its proper type,
+       not to PTR.
+
+       * symfile.c (reread_symbols): Include bfd_errmsg string in error
+       message if bfd_close fails.
+       * exec.c (exec_close), solib.c (clear_solib), corelow.c
+       (core_close), objfiles.c (free_objfile), irix5-nat.c
+       (clear_solib), osfsolib.c (clear_solib), remote-utils.c
+       (gr_load_image): Check for errors from bfd_close.
+       * solib.c (look_for_base), remote-utils.c (gr_load_image),
+       remote-udi.c (download), corelow.c (core_open), symfile.c
+       (symfile_bfd_open), symfile.c (generic_load): Add comment
+       regarding error from bfd_close.
+       * remote-udi.c (download), remote-utils.c (gr_load_image): Add
+       comment about bogus handling of errors from bfd_openr.
+       * exec.c (exec_close): Add comment regarding memory leak and
+       dangling reference to vp->name.
+
+Sat Oct 15 03:43:00 1994  Peter Schauer  (pes@regent.e-technik.tu-muenchen.de)
+
+       * eval.c (evaluate_subexp):  Make fnptr a LONGEST instead
+       of using longest_to_int.
+
+       * infcmd.c (run_stack_dummy):  Reinstate set_current_frame call,
+       mips and alpha targets need the real breakpoint pc for
+       creating the breakpoint frame.
+
+       * stack.c (return_command):  Cast return value to the return
+       type of the function from which we return.
+       * values.c (set_return_value):  Pass VALUE_CONTENTS unmodified
+       to STORE_RETURN_VALUE.
+
+       * symtab.c (lookup_symbol):  Remove search for `static mangled
+       symbols', the search for `static symbols' already looks for
+       mangled and demangled symbols via lookup_block_symbol.
+
+       * valarith.c (value_binop):  Use ANSI C arithmetic conversions
+       when performing integral evaluations, implement BINOP_EQUAL and
+       BINOP_LESS.
+       (value_equal, value_less):  Use value_binop to perform the
+       comparison if both operands have TYPE_CODE_INT.
+
+       * rs6000-tdep.c (pop_frame):  Make sure all registers are valid,
+       as they are written back later. Handle sp restore for frameless
+       functions. Use fdata.nosavedpc instead of fdata.frameless to
+       determine if the pc has been saved.
+       (function_frame_info):  Handle `mr r31,r1', which is generated by
+       gcc-2.6, as a synonym for `oril r31,r1,0'.
+       (skip_trampoline_code):  Handle shared library trampolines.
+       * xcoffread.c (read_xcoff_symtabs):  Record XMC_GL symbols with
+       their real name. Enables setting of breakpoints in shared libraries
+       before the executable is run.
+
+Fri Oct 14 19:39:47 1994  Rob Savoye  <rob@darkstar.cygnus.com>
+
+       * monitor.h, remote-mon.c: Hack up to so the old ROM monitor
+       interface code still works with the new ROM monitor
+       structures. Fake out a couple of fields.
+
+Fri Oct 14 14:54:37 1994  Stan Shebs  (shebs@andros.cygnus.com)
+
+       * h8500-tdep.c (target_read_sp, target_write_sp, target_read_pc,
+       target_write_pc, target_read_fp, target_write_fp): Rename to
+       h8500_read_sp, etc.
+       (h8500_read_pc, h8500_write_pc): Add pid argument.
+       * config/h8500/tm-h8500.h (TARGET_READ_SP, TARGET_WRITE_SP,
+       TARGET_READ_PC, TARGET_WRITE_PC, TARGET_READ_FP, TARGET_WRITE_FP):
+       Change to match functions above.
+
+Thu Oct 13 13:24:29 1994  Jim Kingdon  (kingdon@lioth.cygnus.com)
+
+       * NEWS: Add item about if and while.
+
+       * .gdbinit: Restore `end'; it was not excess.  Reindent
+       list-objfiles to make this clear.  Comment out all of
+       list-objfiles because old gdb's choke on it.
+
+Wed Oct 12 23:19:08 1994  Ian Lance Taylor  <ian@sanguine.cygnus.com>
+
+       * config/mips/tm-bigmips64.h: Just define TARGET_BYTE_ORDER and
+       include tm-mips64.h.
+
+Wed Oct 12 18:02:17 1994  Stan Shebs  (shebs@andros.cygnus.com)
+
+       * Makefile.in (ANNOTATE_OBS): New definition.
+       (COMMON_OBS): Add exec.o.
+       (annotate.o): Remove extra compile rule.
+       * config/*/*.mh, config/*/*.mt: Remove exec.o from *DEPFILES lists
+       everywhere.
+
+       * .gdbinit: Remove excess `end'.
+
+       * exec.c: Merge in RS6000 support from xcoffexec.c.
+       (symfile.h, objfiles.h, xcoffsolib.h): Include.
+       (vmap): New global variable.
+       (exec_close): Close and free objects in vmap chain.
+       (exec_file_command) [IBM6000_TARGET]: Set up initial vmap.
+       (bfdsec_to_vmap, map_vmap): Moved here from xcoffexec.c.
+       (exec_files_info): Print vmap information.
+       * xcoffexec.c: Remove.
+       * config/rs6000/rs6000.mt, config/rs6000/rs6000lynx.mt
+       (TDEPFILES): Use exec.o instead of xcoffexec.o.
+       * TODO: Remove pertinent items.
+
+Wed Oct 12 10:08:19 1994  Jeff Law  (law@snake.cs.utah.edu)
+
+       * partial-stab.h (N_TEXT): Delete GDB_TARGET_IS_HPPA kludge; they
+       are no longer needed as of gcc-2.6.0.
+
+Tue Oct 11 15:51:01 1994  Ian Lance Taylor  <ian@sanguine.cygnus.com>
+
+       * lynx-nat.c (child_wait): Correct handling of byte reversed SPARC
+        Lynx wait status.
+       (fetch_core_registers): Don't try to fetch a register if
+       regmap maps it to -1.
+       * sparc-tdep.c (sparc_frame_find_saved_regs): Use FRAME_SAVED_I0
+       and FRAME_SAVED_L0 when setting saved_regs_addr.  SPARC Lynx
+       stores the registers in a weird order.
+
+Sat Oct  8 20:59:13 1994  Jim Kingdon  (kingdon@lioth.cygnus.com)
+
+       * blockframe.c (reinit_frame_cache): Reinstate select_frame call
+       if inferior_pid is nonzero.
+
+Sat Oct  8 04:27:21 1994  Peter Schauer  (pes@regent.e-technik.tu-muenchen.de)
+
+       Speed up GDB startup time by not demangling partial symbols.
+       * symfile.h (ADD_PSYMBOL_VT_TO_LIST),
+       symfile.c (add_psymbol_to_list, add_psymbol_addr_to_list):
+       No longer demangle partial symbols.
+       * symtab.c (lookup_symbol, list_symbols): Handle mangled
+       variables, e.g. C++ static members, via the minimal symbols.
+
+       Handle reordered functions in an objfile, for Irix 5.2 shared
+       libraries.
+       * objfiles.h (OBJF_REORDERED):  New bit in the objfile flags,
+       set if the functions in an objfile are reordered.
+       * mdebugread.c (parse_partial_symbols):  Detect reordered
+       functions in an objfile.
+       * symtab.c (find_pc_psymtab, find_pc_symtab):  Use expensive
+       lookup algorithm if the functions in the objfile are reordered.
+
+       * xcoffexec.c (exec_close):  If the current target has a copy
+       of the exec_ops sections, reflect the freeing of the sections
+       in current_target.
+
+       * valops.c (call_function_by_hand):  Use `sizeof dummy1', not
+       `sizeof dummy', for constructing the call dummy code.
+
+       * config/sparc/tm-sparc.h:  Add PARAMS declarations to all
+       function declarations.
+       * sparc-tdep.c (sparc_pop_frame):  Cast result of
+       read_memory_integer to CORE_ADDR when passing it to PC_ADJUST.
+
+       * irix5-nat.c (enable_break):  Set breakpoint at the entry point
+       of the executable, to handle the case where main resides in a
+       shared library.
+       * irix5-nat.c (solib_create_inferior_hook):  Reset stop_soon_quietly
+       after shared library symbol reading, to get rid of a warning from
+       heuristic_proc_start if the startup code has no symbolic debug info.
+
+       * breakpoint.h (struct breakpoint):  Add new fields language
+       and input_radix, to enable breakpoint resetting with the
+       proper language and radix.
+       * breakpoint.c (set_raw_breakpoint):  Initialize them.
+       (breakpoint_re_set_one):  Use them when resetting the breakpoint.
+       (breakpoint_re_set):  Preserve current language and input_radix
+       across breakpoint_re_set_one calls.
+
+       * symtab.c (decode_line_1):  Do not build a canonical line
+       specification for `*expr' line specifications.
+
+       * breakpoint.h (bpstat_stop_status):  Fix prototype declaration.
+
+Fri Oct  7 08:48:18 1994  Jim Kingdon  (kingdon@lioth.cygnus.com)
+
+       The point of these changes is to avoid reading the frame pointer
+       and stack pointer during stepping, to speed things up.
+       A.  Changes to not select a frame until we need a selected frame:
+           * blockframe.c (flush_cached_frames): Call select_frame (NULL, -1).
+           * infrun.c (wait_for_inferior): Move call to select_frame back to
+           normal_stop.  This reverts a change of 13 Apr 94 (it says Jeff
+           Law, but the change was my idea); the only reason for that change
+           was so we could save and restore the selected frame in
+           wait_for_inferior, and now that flush_cached frames clears the
+           selected frame, that should work OK now.
+       B.  Changes to not create a current_frame until we need one:
+           * blockframe.c (get_current_frame): If current_frame is NULL, try
+           to create an innermost frame.
+           * sparc-tdep.c (sparc_pop_frame), infcmd.c (run-stack_dummy),
+           infrun.c (wait_for_inferior), thread.c (thread_switch),
+           convex-tdep.c (set_thread_command), a29k-tdep.c (pop_frame),
+           alpha-tdep.c (alpha_pop_frame), convex-xdep.c (core_file_command),
+           h8300-tdep.c (h8300_pop_frame), h8500-tdep.c (h8300_pop_frame),
+           hppa-tdep.c (hppa_pop_frame), i386-tdep.c (i386_pop_frame),
+           i960-tdep.c (pop_frame), m68k-tdep.c
+           (m68k_pop_frame), mips-tdep.c (mips_pop_frame), rs6000-tdep.c
+           (push_dummy_frame, pop_dummy_frame, pop_frame), sh-tdep.c
+           (pop_frame), config/arm/tm-arm.h (POP_FRAME),
+           config/convex/tm-convex.h (POP_FRAME), config/gould/tm-pn.h
+           (POP_FRAME), config/ns32k/tm-merlin.h (POP_FRAME),
+           config/ns32k/tm-umax.h (POP_FRAME), config/tahoe/tm-tahoe.h
+           (POP_FRAME), config/vax/tm-vax.h (POP_FRAME): Don't
+           call create_new_frame.
+           * corelow.c (core_open), altos-xdep.c (core_file_command),
+           arm-xdep.c (core_file_command), gould-xdep.c (core_file_command),
+           m3-nat.c (select_thread), sun386-nat.c (core_file_command),
+           umax-xdep.c (core_file_command): Don't call create_new_frame; do
+           call flush_cached_frames.
+           * blockframe.c (reinit_frame_cache): Don't call create_new_frame
+           or select_frame.
+       C.  Changes to get rid of stop_frame_address and instead only
+       fetch the frame pointer when we need it.
+           * breakpoint.c (bpstat_stop_status): Remove argument
+           frame_address; use FRAME_FP (get_current_frame ()).
+           * infrun.c (wait_for_inferior): Don't pass frame pointer to
+           bpstat_stop_status.
+           * infrun.c (wait_for_inferior): Use FRAME_FP (get_current_frame
+           ()) instead of stop_frame_address.
+           * infrun.c (save_inferior_status, restore_inferior_status),
+           inferior.h (struct inferior_status): Don't save and restore
+           stop_frame_address.
+           * inferior.h, infcmd.c, thread.c (thread_switch), m3-nat.c
+           (select_thread): Remove stop_frame_address and uses thereof.
+       D.  Same thing for the stack pointer.
+           * infrun.c (wait_for_inferior): Remove stop_sp and replace
+           uses thereof with read_sp ().
+       E.  Change to eliminate one nasty little spot where we were
+       wanting to know the frame pointer from before the current step
+       (idea from GDB 3.5, which saved my ass, because my other ideas of
+       how to fix it were very baroque).
+           * infrun.c: Remove prev_frame_address.
+           * infrun.c (wait_for_inferior, step_over_function): Use
+           step_frame_address instead of prev_frame_address.
+       F.  Same basic idea for the stack pointer.
+           * inferior.h, infcmd.c: New variable step_sp.
+           * infcmd.c (step_1, until_next_command): Set it.
+           * infrun.c: Remove prev_sp and replace uses by step_sp.
+           * infrun.c (wait_for_inferior): If we get out of the step
+           range, then set step_sp to the current stack pointer before we
+           start going again.
+
+Fri Oct  7 12:17:17 1994  Ian Lance Taylor  <ian@sanguine.cygnus.com>
+
+       * top.c (target_byte_order_auto): New static variable.
+       (set_endian): Mention that ``auto'' is permitted.
+       (set_endian_auto): New static function.
+       (show_endian): Change message based on target_byte_order_auto.
+       (set_endian_from_file): New function.
+       (init_main): Add command ``auto'' to endianlist.
+       * exec.c (exec_file_command): Call set_endian_from_file.
+       * defs.h (set_endian_from_file): Declare.
+
+Thu Oct  6 18:10:41 1994  J.T. Conklin  (jtc@phishhead.cygnus.com)
+
+       * nlm/i386.c (flush_i_cache): New function, does nothing.
+       (frame_to_registers, registers_to_frame, set_step_traps,
+       clear_step_traps, do_status): Make non-static.
+
+Thu Oct  6 12:26:42 1994  Ian Lance Taylor  <ian@sanguine.cygnus.com>
+
+       * config/mips/tm-mips.h (GDB_TARGET_IS_MIPS64): If not already
+       defined, define as 0.
+       (FIX_CALL_DUMMY): Rewrite to remove presumption that host and
+       target are similar.
+       * config/mips/tm-idt.h (TARGET_BYTE_ORDER_SELECTABLE): Define.
+       * config/mips/tm-idtl.h (TARGET_BYTE_ORDER_SELECTABLE): Define.
+       * config/mips/tm-idt64.h (TARGET_BYTE_ORDER_SELECTABLE): Define.
+       (BREAKPOINT): Remove definition.
+       * config/mips/tm-idtl64.h (TARGET_BYTE_ORDER_SELECTABLE): Define.
+       (BREAKPOINT): Remove definition.
+       * config/mips/tm-mips64.h (GDB_TARGET_IS_MIPS64): Define with a
+       value of 1, rather than without a value.
+       * config/mips/tm-bigmips64.h (GDB_TARGET_IS_MIPS64): Likewise.
+       * mips-tdep.c: Rewrite uses of GDB_TARGET_IS_MIPS64 to switch at
+       run time rather than at compile time.
+
+       * remote-mips.c (break_insn): Remove.
+       (BREAK_INSN, BREAK_INSN_SIZE): Define.
+       (mips_insert_breakpoint): Use BREAK_INSN, not break_insn.
+       (mips_remove_breakpoint): Likewise.
+
+       * defs.h: If TARGET_BYTE_ORDER_SELECTABLE is defined by tm.h,
+       define TARGET_BYTE_ORDER as target_byte_order, and declare
+       target_byte_order as an extern int, and define BITS_BIG_ENDIAN as
+       a test of TARGET_BYTE_ORDER.
+       * top.c: Several additions if TARGET_BYTE_ORDER_SELECTABLE is
+       defined:
+       (endianlist, target_byte_order): New variables.
+       (set_endian, set_endian_big, set_endian_little): New functions.
+       (show_endian): New function.
+       (init_cmd_lists): Initialize endianlist.
+       (init_main): Add commands ``set endian big'', ``set endian
+       little'', and ``show endian''.
+       * a29k-pinsn.c: Rewrite uses of TARGET_BYTE_ORDER and
+       BITS_BIG_ENDIAN to switch at run time rather than at compile time.
+       * coffread.c, dwarfread.c, findvar.c, mips-tdep.c: Likewise.
+       * remote-os9k.c, stabsread.c, valarith.c, valprint.c: Likewise.
+       * values.c: Likewise.
+
+Wed Oct  5 11:41:24 1994  J.T. Conklin  (jtc@phishhead.cygnus.com)
+
+       * nlm/configure.in: ${gdb_host_cpu} defaults to ${host_cpu}.
+
+       * nlm/Makefile.in: Get rid of NWINCLUDES.
+       * config/{alpha,powerpc}/gdbserve.mt: Remove NWINCLUDES.
+         User should now configure with --with-headers.
+
+Mon Oct  3 07:48:34 1994  Jim Kingdon  (kingdon@lioth.cygnus.com)
+
+       * gdbserver/server.c (main): Silently accept all unrecognized
+       requests and send back a zero length acknowledge.  That is what
+       *-stub.c do and is what remote.c expects.
+
+Mon Oct  3 05:11:47 1994  Peter Schauer  (pes@regent.e-technik.tu-muenchen.de)
+
+       * corelow.c (core_open):  Copy the modified to_sections_end
+       vector from current_target to core_ops too.
+
+       * gdbserver/server.c (main):  Silently accept query requests
+       and send back a zero length acknowledge.
+
+Fri Sep 30 17:17:21 1994  J.T. Conklin  (jtc@phishhead.cygnus.com)
+
+        * nlm/Makefile.in: Don't define NWINCLUDES.
+        * config/{alpha,powerpc}/gdbserve.mt: define NWINCLUDES.
+
+Fri Sep 30 15:59:55 1994  Jim Kingdon  (kingdon@lioth.cygnus.com)
+
+       * gdbserver/low-lynx.c (create_inferior): Pass all 4 args to ptrace.
+
+Fri Sep 30 06:42:42 1994  Ian Lance Taylor  (ian@cygnus.com)
+
+       * lynx-nat.c (child_wait): Use status.w_status, not status, in
+       arithmetic.  status is a `union wait'.
+
+       * config/nm-lynx.h (PTRACE_ARG3_TYPE): Define to int, not char *.
+
+       * lynx-nat.c (child_wait): Pass fourth argument to ptrace.
+
 Thu Sep 29 08:22:27 1994  Jim Kingdon  (kingdon@lioth.cygnus.com)
 
        * xcoffread.c (read_xcoff_symtab): Fix comment for yesterday's change.
This page took 0.027033 seconds and 4 git commands to generate.