gdbserver: include aarch32/aarch64 header file in corresponding source file
authorSimon Marchi <simon.marchi@polymtl.ca>
Sun, 12 Jan 2020 16:06:21 +0000 (11:06 -0500)
committerSimon Marchi <simon.marchi@polymtl.ca>
Sun, 12 Jan 2020 16:06:38 +0000 (11:06 -0500)
When building gdbserver for an aarch64 host with -Wmissing-declarations,
I see:

  CXX    linux-aarch32-tdesc.o
/home/simark/src/binutils-gdb/gdb/gdbserver/linux-aarch32-tdesc.c:28:1: error: no previous declaration for 'const target_desc* aarch32_linux_read_description()' [-Werror=missing-declarations]
 aarch32_linux_read_description ()
 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/home/simark/src/binutils-gdb/gdb/gdbserver/linux-aarch32-tdesc.c:43:1: error: no previous declaration for 'bool is_aarch32_linux_description(const target_desc*)' [-Werror=missing-declarations]
 is_aarch32_linux_description (const target_desc *tdesc)
 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~

  CXX    linux-aarch64-tdesc.o
/home/simark/src/binutils-gdb/gdb/gdbserver/linux-aarch64-tdesc.c:32:1: error: no previous declaration for 'const target_desc* aarch64_linux_read_description(uint64_t, bool)' [-Werror=missing-declarations]
 aarch64_linux_read_description (uint64_t vq, bool pauth_p)
 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Fix it by including linux-aarch32-tdesc.h in linux-aarch32-tdesc.c and
linux-aarch64-tdesc.h in linux-aarch64-tdesc.c.

gdb/gdbserver/ChangeLog:

* linux-aarch32-tdesc.c: Include linux-aarch32-tdesc.h.
* linux-aarch64-tdesc.c: Include linux-aarch64-tdesc.h.

gdb/gdbserver/ChangeLog
gdb/gdbserver/linux-aarch32-tdesc.c
gdb/gdbserver/linux-aarch64-tdesc.c

index b62ed4c02b47e8bcee507045e566419c571a4b3b..40a765a3eded04cece7be43f6ff77030a1073f58 100644 (file)
@@ -1,3 +1,8 @@
+2020-01-12  Simon Marchi  <simon.marchi@polymtl.ca>
+
+       * linux-aarch32-tdesc.c: Include linux-aarch32-tdesc.h.
+       * linux-aarch64-tdesc.c: Include linux-aarch64-tdesc.h.
+
 2020-01-10  Pedro Alves  <palves@redhat.com>
 
        * fork-child.c (post_fork_inferior): Pass target down to
index 98d9ac8fd95ebd91afbb3ac479323f0eeba45280..b0dffe27e7114d8f5dbeb86b70900d807292603c 100644 (file)
@@ -16,6 +16,9 @@
    along with this program.  If not, see <http://www.gnu.org/licenses/>.  */
 
 #include "server.h"
+
+#include "linux-aarch32-tdesc.h"
+
 #include "tdesc.h"
 #include "arch/aarch32.h"
 #include <inttypes.h>
index 85fe4143d6ade5632e22468d5ba1d8538bbc41e3..897fbb43bd28ddf44c69d4162dda43c2589b060f 100644 (file)
@@ -18,6 +18,9 @@
    along with this program.  If not, see <http://www.gnu.org/licenses/>.  */
 
 #include "server.h"
+
+#include "linux-aarch64-tdesc.h"
+
 #include "tdesc.h"
 #include "arch/aarch64.h"
 #include "linux-aarch32-low.h"