+2015-08-11 Keith Seitz <keiths@redhat.com>
+
+ * breakpoint.c (create_thread_event_breakpoint, init_breakpoint_sal):
+ Convert linespec to address location.
+ * linespec.c (canonicalize_linespec): Do not handle address
+ locations here.
+ (convert_address_location_to_sals): New function; contents moved
+ from ...
+ (convert_linespc_to_sals): ... here.
+ (parse_linespec): Remove address locations from linespec grammar.
+ Remove handling of address locations.
+ (linespec_lex_to_end): Remove handling of address linespecs.
+ (event_location_to_sals): Handle ADDRESS_LOCATION.
+ (linespec_expression_to_pc): Export.
+ * linespec.h (linespec_expression_to_pc): Add declaration.
+ * location.c (struct event_location.u) <address>: New member.
+ (new_address_location, get_address_location): New functions.
+ (copy_event_location, delete_event_location, event_location_to_string)
+ (string_to_event_location, event_location_empty_p): Handle address
+ locations.
+ * location.h (enum event_location_type): Add ADDRESS_LOCATION.
+ (new_address_location, get_address_location): Declare.
+ * python/py-finishbreakpoint.c (bpfinishpy_init): Convert linespec
+ to address location.
+ * spu-tdep.c (spu_catch_start): Likewise.
+
+2015-08-11 Keith Seitz <keiths@redhat.com>
+
+ * ax-gdb.c: Include location.h.
+ (agent_command_1) Use linespec location instead of address
+ string.
+ * break-catch-throw.c: Include location.h.
+ (re_set_exception_catchpoint): Use linespec locations instead
+ of address strings.
+ * breakpoint.c: Include location.h.
+ (create_overlay_event_breakpoint, create_longjmp_master_breakpoint)
+ (create_std_terminate_master_breakpoint)
+ (create_exception_master_breakpoint, update_breakpoints_after_exec):
+ Use linespec location instead of address string.
+ (print_breakpoint_location): Use locations and
+ event_location_to_string.
+ Print extra_string for pending locations for non-MI streams.
+ (print_one_breakpoint_location): Use locations and
+ event_location_to_string.
+ (init_raw_breakpoint_without_location): Initialize b->location.
+ (create_thread_event_breakpoint): Use linespec location instead of
+ address string.
+ (init_breakpoint_sal): Likewise.
+ Only save extra_string if it is non-NULL and not the empty string.
+ Use event_location_to_string instead of `addr_string'.
+ Constify `p' and `endp'.
+ Use skip_spaces_const/skip_space_const instead of non-const versions.
+ Copy the location into the breakpoint.
+ If LOCATION is NULL, save the breakpoint address as a linespec location
+ instead of an address string.
+ (create_breakpoint_sal): Change `addr_string' parameter to a struct
+ event_location. All uses updated.
+ (create_breakpoints_sal): Likewise for local variable `addr_string'.
+ (parse_breakpoint_sals): Use locations instead of address strings.
+ Remove check for empty linespec with conditional.
+ Refactor.
+ (decode_static_tracepoint_spec): Make argument const and update
+ function.
+ (create_breakpoint): Change `arg' to a struct event_location and
+ rename.
+ Remove `copy_arg' and `addr_start'.
+ If EXTRA_STRING is empty, set it to NULL.
+ Don't populate `canonical' for pending breakpoints.
+ Pass `extra_string' to find_condition_and_thread.
+ Clear `extra_string' if `rest' was NULL.
+ Do not error with "garbage after location" if setting a dprintf
+ breakpoint.
+ Copy the location into the breakpoint instead of an address string.
+ (break_command_1): Use string_to_event_location and pass this to
+ create_breakpoint instead of an address string.
+ Check against `arg_cp' for a probe linespec.
+ (dprintf_command): Use string_to_event_location and pass this to
+ create_breakpoint instead of an address string.
+ Throw an exception if no format string was specified.
+ (print_recreate_ranged_breakpoint): Use event_location_to_string
+ instead of address strings.
+ (break_range_command, until_break_command)
+ (init_ada_exception_breakpoint): Use locations instead
+ of address strings.
+ (say_where): Print out extra_string for pending locations.
+ (base_breakpoint_dtor): Delete `location' and `location_range_end' of
+ the breakpoint.
+ (base_breakpoint_create_sals_from_location): Use struct event_location
+ instead of address string.
+ Remove `addr_start' and `copy_arg' parameters.
+ (base_breakpoint_decode_location): Use struct event_location instead of
+ address string.
+ (bkpt_re_set): Use locations instead of address strings.
+ Use event_location_empty_p to check for unset location.
+ (bkpt_print_recreate): Use event_location_to_string instead of
+ an address string.
+ Print out extra_string for pending locations.
+ (bkpt_create_sals_from_location, bkpt_decode_location)
+ (bkpt_probe_create_sals_from_location): Use struct event_location
+ instead of address string.
+ (bkpt_probe_decode_location): Use struct event_location instead of
+ address string.
+ (tracepoint_print_recreate): Use event_location_to_string to
+ recreate the tracepoint.
+ (tracepoint_create_sals_from_location, tracepoint_decode_location)
+ (tracepoint_probe_create_sals_from_location)
+ (tracepoint_probe_decode_location): Use struct event_location
+ instead of address string.
+ (dprintf_print_recreate): Use event_location_to_string to recreate
+ the dprintf.
+ (dprintf_re_set): Remove check for valid/missing format string.
+ (strace_marker_create_sals_from_location)
+ (strace_marker_create_breakpoints_sal, strace_marker_decode_location)
+ (update_static_tracepoint): Use struct event_location instead of
+ address string.
+ (location_to_sals): Likewise.
+ Pass `extra_string' to find_condition_and_thread.
+ For newly resolved pending breakpoint locations, clear the location's
+ string representation.
+ Assert that the breakpoint's condition string is NULL when
+ condition_not_parsed.
+ (breakpoint_re_set_default, create_sals_from_location_default)
+ (decode_location_default, trace_command, ftrace_command)
+ (strace_command, create_tracepoint_from_upload): Use locations
+ instead of address strings.
+ * breakpoint.h (struct breakpoint_ops) <create_sals_from_location>:
+ Use struct event_location instead of address string.
+ Update all uses.
+ <decode_location>: Likewise.
+ (struct breakpoint) <addr_string>: Change to struct event_location
+ and rename `location'.
+ <addr_string_range_end>: Change to struct event_location and rename
+ `location_range_end'.
+ (create_breakpoint): Use struct event_location instead of address
+ string.
+ * cli/cli-cmds.c: Include location.h.
+ (edit_command, list_command): Use locations instead of address strings.
+ * elfread.c: Include location.h.
+ (elf_gnu_ifunc_resolver_return_stop): Use event_location_to_string.
+ * guile/scm-breakpoint.c: Include location.h.
+ (bpscm_print_breakpoint_smob): Use event_location_to_string.
+ (gdbscm_register_breakpoint): Use locations instead of address
+ strings.
+ * linespec.c: Include location.h.
+ (struct ls_parser) <stream>: Change to const char *.
+ (PARSER_STREAM): Update.
+ (lionespec_lexer_lex_keyword): According to find_condition_and_thread,
+ keywords must be followed by whitespace.
+ (canonicalize_linespec): Save a linespec location into `canonical'.
+ Save a canonical linespec into `canonical'.
+ (parse_linespec): Change `argptr' to const char * and rename `arg'.
+ All uses updated.
+ Update function description.
+ (linespec_parser_new): Initialize `parser'.
+ Update initialization of parsing stream.
+ (event_location_to_sals): New function.
+ (decode_line_full): Change `argptr' to a struct event_location and
+ rename it `location'.
+ Use locations instead of address strings.
+ Call event_location_to_sals instead of parse_linespec.
+ (decode_line_1): Likewise.
+ (decode_line_with_current_source, decode_line_with_last_displayed)
+ Use locations instead of address strings.
+ (decode_objc): Likewise.
+ Change `argptr' to const char * and rename `arg'.
+ (destroy_linespec_result): Delete the linespec result's location
+ instead of freeing the address string.
+ * linespec.h (struct linespec_result) <addr_string>: Change to
+ struct event_location and rename to ...
+ <location>: ... this.
+ (decode_line_1, decode_line_full): Change `argptr' to struct
+ event_location. All callers updated.
+ * mi/mi-cmd-break.c: Include language.h, location.h, and linespec.h.
+ (mi_cmd_break_insert_1): Use locations instead of address strings.
+ Throw an error if there was "garbage" at the end of the specified
+ linespec.
+ * probe.c: Include location.h.
+ (parse_probes): Change `argptr' to struct event_location.
+ Use event locations instead of address strings.
+ * probe.h (parse_probes): Change `argptr' to struct event_location.
+ * python/py-breakpoint.c: Include location.h.
+ (bppy_get_location): Constify local variable `str'.
+ Use event_location_to_string.
+ (bppy_init): Use locations instead of address strings.
+ * python/py-finishbreakpoint.c: Include location.h.
+ (bpfinishpy_init): Remove local variable `addr_str'.
+ Use locations instead of address strings.
+ * python/python.c: Include location.h.
+ (gdbpy_decode_line): Use locations instead of address strings.
+ * remote.c: Include location.h.
+ (remote_download_tracepoint): Use locations instead of address
+ strings.
+ * spu-tdep.c: Include location.h.
+ (spu_catch_start): Remove local variable `buf'.
+ Use locations instead of address strings.
+ * tracepoint.c: Include location.h.
+ (scope_info): Use locations instead of address strings.
+ (encode_source_string): Constify parameter `src'.
+ * tracepoint.h (encode_source_string): Likewise.
+
+2015-08-11 Keith Seitz <keiths@redhat.com>
+
+ * Makefile.in (SFILES): Add location.c.
+ (HFILES_NO_SRCDIR): Add location.h.
+ (COMMON_OBS): Add location.o.
+ * linespec.c (linespec_lex_to_end): New function.
+ * linespec.h (linespec_lex_to_end): Declare.
+ * location.c: New file.
+ * location.h: New file.
+
+2015-08-11 Keith Seitz <keiths@redhat.com>
+
+ * breakpoint.h (struct breakpoint_ops) <create_sals_from_address>:
+ Renamed to create_sals_from_location.
+ <decode_linespec>: Renamed to decode_location.
+ Update all callers.
+ * breakpoint.c (create_sals_from_address_default): Renamed to ...
+ (create_sals_from_location_default): ... this.
+ (addr_string_to_sals): Renamed to ...
+ (location_to_sals): ... this.
+ (decode_linespec_default): Renamed to ...
+ (decode_location_default): ... this.
+ (base_breakpoint_create_sals_from_address): Renamed to ...
+ (base_breakpoint_create_sals_from_location): ... this.
+ (bkpt_create_sals_from_address): Renamed to ...
+ (bkpt_create_sals_from_location): ... this.
+ (bkpt_decode_linespec): Renamed to ...
+ (bkpt_decode_location): ... this.
+ (bkpt_probe_create_sals_from_address): Renamed to ...
+ (bkpt_probe_create_sals_from_location): ... this.
+ (tracepoint_create_sals_from_address): Renamed to ...
+ (tracepoint_create_sals_from_location): ... this.
+ (tracepoint_decode_linespec): Renamed to ...
+ (tracepoint_decode_location): ... this.
+ (tracepoint_probe_create_sals_from_address): Renamed to ...
+ (tracepoint_probe_create_sals_from_location): ... this.
+ (tracepoint_probe_decode_linespec): Renamed to ...
+ (tracepoint_probe_decode_location): ... this.
+ (strace_marker_create_sals_from_address): Renamed to ...
+ (strace_marker_create_sals_from_location): ... this.
+ (decode_linespec_default): Renamed to ...
+ (decode_location_default): ... this.
+
+2015-08-10 Doug Evans <dje@google.com>
+ Keith Seitz <keiths@redhat.com>
+
+ PR gdb/17960
+ * symtab.c (make_file_symbol_completion_list_1): Renamed from
+ make_file_symbol_completion_list and made static.
+ (make_file_symbol_completion_list): New function.
+
+2015-08-10 Joel Brobecker <brobecker@adacore.com>
+
+ * infrun.c (follow_fork, displaced_step_prepare, resume): Remove
+ trailing new-line at end of warning message.
+ (proceed): Add i18n marker to error messages.
+
+2015-08-07 Pedro Alves <palves@redhat.com>
+
+ * linux-nat.c (linux_nat_always_non_stop_p): Return 1.
+
+2015-08-07 Pedro Alves <palves@redhat.com>
+
+ * s390-linux-tdep.c (is_non_branch_ril)
+ (s390_displaced_step_copy_insn): New functions.
+ (s390_displaced_step_fixup): Update comment.
+ (s390_gdbarch_init): Install s390_displaced_step_copy_insn as
+ gdbarch_displaced_step_copy_insn hook.
+
+2015-08-07 Pedro Alves <palves@redhat.com>
+
+ * infrun.c (displaced_step_prepare_throw): Return -1 if
+ gdbarch_displaced_step_copy_insn returns NULL. Update intro
+ comment.
+ * rs6000-tdep.c (LWARX_MASK, LWARX_INSTRUCTION, LDARX_INSTRUCTION)
+ (STWCX_MASK, STWCX_INSTRUCTION, STDCX_INSTRUCTION): Move higher up
+ in file.
+ (ppc_displaced_step_copy_insn): New function.
+ (ppc_displaced_step_fixup): Update comment.
+ (rs6000_gdbarch_init): Install ppc_displaced_step_copy_insn as
+ gdbarch_displaced_step_copy_insn hook.
+ * gdbarch.sh (displaced_step_copy_insn): Document what happens on
+ NULL return.
+ * gdbarch.h: Regenerate.
+
+2015-08-07 Pedro Alves <palves@redhat.com>
+
+ * inferior.h (struct inferior) <displaced_stepping_failed>: New
+ field.
+ * infrun.c (use_displaced_stepping_now_p): New parameter 'inf'.
+ Return false if dispaced stepping failed before.
+ (resume): Pass the current inferior to
+ use_displaced_stepping_now_p. Wrap displaced_step_prepare in
+ TRY/CATCH. If we get a MEMORY_ERROR, set the inferior's
+ displaced_stepping_failed flag, and fall back to an in-line
+ step-over.
+
+2015-08-07 Pedro Alves <palves@redhat.com>
+
+ * darwin-nat.c (darwin_stop): Rename to ...
+ (darwin_interrupt): ... this.
+ (_initialize_darwin_inferior): Adjust.
+ * gnu-nat.c (gnu_stop): Delete.
+ (gnu_target): Don't install gnu_stop.
+ * inf-ptrace.c (inf_ptrace_stop): Rename to ...
+ (inf_ptrace_interrupt): ... this.
+ (inf_ptrace_target): Adjust.
+ * infcmd.c (interrupt_target_1): Use target_interrupt instead of
+ target_stop.
+ * linux-nat (linux_nat_stop): Rename to ...
+ (linux_nat_interrupt): ... this.
+ (linux_nat_stop): Reimplement.
+ (linux_nat_add_target): Install linux_nat_interrupt.
+ * nto-procfs.c (nto_interrupt_twice): Rename to ...
+ (nto_handle_sigint_twice): ... this.
+ (nto_interrupt): Rename to ...
+ (nto_handle_sigint): ... this. Call target_interrupt instead of
+ target_stop.
+ (procfs_wait): Adjust.
+ (procfs_stop): Rename to ...
+ (procfs_interrupt): ... this.
+ (init_procfs_targets): Adjust.
+ * procfs.c (procfs_stop): Rename to ...
+ (procfs_interrupt): ... this.
+ (procfs_target): Adjust.
+ * remote-m32r-sdi.c (m32r_stop): Rename to ...
+ (m32r_interrupt): ... this.
+ (init_m32r_ops): Adjust.
+ * remote-sim.c (gdbsim_stop_inferior): Rename to ...
+ (gdbsim_interrupt_inferior): ... this.
+ (gdbsim_stop): Rename to ...
+ (gdbsim_interrupt): ... this.
+ (gdbsim_cntrl_c): Adjust.
+ (init_gdbsim_ops): Adjust.
+ * remote.c (sync_remote_interrupt): Adjust comments.
+ (remote_stop_as): Rename to ...
+ (remote_interrupt_as): ... this.
+ (remote_stop): Adjust comment.
+ (remote_interrupt): New function.
+ (init_remote_ops): Install remote_interrupt.
+ * target.c (target_interrupt): New function.
+ * target.h (struct target_ops) <to_interrupt>: New field.
+ (target_interrupt): New declaration.
+ * windows-nat.c (windows_stop): Rename to ...
+ (windows_interrupt): ... this.
+ * target-delegates.c: Regenerate.
+
+2015-08-07 Pedro Alves <palves@redhat.com>
+
+ * signal-while-stepping-over-bp-other-thread.exp: Expect "restart
+ threads" as alternative to "switching back to stepped thread".
+
2015-08-07 Pedro Alves <palves@redhat.com>
* NEWS: Mention "maint set/show target-non-stop".