Re: [PATCH v3 07/12] btrace: Remove struct btrace_thread_info::{begin,end}.

[Simon Marchi <simon dot marchi at polymtl dot ca>]

On 2017-05-09 02:55, Tim Wiederhake wrote:
> These are no longer needed and might hold invalid addresses once we 
> change the
> vector of function segment pointers into a vector of function segment 
> objects
> where a reallocation of the vector changes the address of its elements.
>
> 2017-05-09  Tim Wiederhake  <tim.wiederhake@intel.com>
>
> gdb/ChangeLog:
>
> 	* btrace.c (ftrace_new_function, ftrace_new_call, ftrace_new_tailcall,
> 	ftrace_new_return, ftrace_new_switch, ftrace_new_gap,
> 	ftrace_update_function, ftrace_compute_global_level_offset,
> 	btrace_compute_ftrace_bts, ftrace_add_pt, btrace_compute_ftrace_pt,
> 	btrace_stitch_bts, btrace_fetch, btrace_clear, btrace_insn_number,
> 	btrace_insn_end, btrace_is_empty): Remove references to
> 	btrace_thread_info::begin and btrace_thread_info::end.
> 	* btrace.h (struct btrace_thread_info): Remove BEGIN and END.
> 	* record-btrace.c (record_btrace_start_replaying): Remove reference to
> 	btrace_thread_info::begin.

LGTM.