diff options
author | Simon Marchi <simon.marchi@efficios.com> | 2020-01-13 14:05:32 -0500 |
---|---|---|
committer | Simon Marchi <simon.marchi@efficios.com> | 2020-01-13 14:05:32 -0500 |
commit | 6e37c371af4a25d28623b618e30f93b040b4d4ae (patch) | |
tree | 13b5d5b60eadbbeeb69fb9b7c5dbda6791a2d80d /gdb/rs6000-nat.c | |
parent | df4a0200dc896abe32bef4172ca047465caaedfe (diff) | |
download | fsf-binutils-gdb-6e37c371af4a25d28623b618e30f93b040b4d4ae.zip fsf-binutils-gdb-6e37c371af4a25d28623b618e30f93b040b4d4ae.tar.gz fsf-binutils-gdb-6e37c371af4a25d28623b618e30f93b040b4d4ae.tar.bz2 |
gdbserver: set IP_AGENT_EXPORT_FUNC to static when not building IPA, add declarations
Fixing the -Wmissing-declarations errors in gdbserver's tracepoint.c is
a bit tricky, because some functions are compiled for both gdbserver, in
which case they should be static, since they are only used in that file,
and for libinproctrace.so, in which case they should be externally
visible, since they need to be looked up. In the case where they are
externally visible, -Wmissing-declarations requires that a declaration
exists (that's the point of the warning).
I've reused the IP_AGENT_EXPORT_FUNC macro to mark the functions as
static when compiled for gdbserver. Some seemingly unnecessary
declarations are added for when compiling libinproctrace.so (thanks to
Tom for the suggestion).
gdb/gdbserver/ChangeLog:
* tracepoint.h (IP_AGENT_EXPORT_FUNC) [!IN_PROCESS_AGENT]:
Define to static.
* tracepoint.c (stop_tracing, flush_trace_buffer,
about_to_request_buffer_space, get_trace_state_variable_value,
set_trace_state_variable_value, gdb_collect): Add declaration.
Change-Id: If9c66151bd00c3b9c5caa27a7c21c5a3a952de2a
Diffstat (limited to 'gdb/rs6000-nat.c')
0 files changed, 0 insertions, 0 deletions