[binutils-gdb] gdbserver: include linux-arm-tdesc.h in linux-arm-tdesc.c

[Simon Marchi]

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=c0b0a14290e1909a6cc3febb443560c3e95562c2

commit c0b0a14290e1909a6cc3febb443560c3e95562c2
Author: Simon Marchi <simon.marchi@polymtl.ca>
Date:   Sun Jan 12 12:07:30 2020 -0500

    gdbserver: include linux-arm-tdesc.h in linux-arm-tdesc.c
    
    When building with -Wmissing-declarations, I get:
    
      CXX    linux-arm-tdesc.o
    /binutils-gdb/gdb/gdbserver/linux-arm-tdesc.c:29:1: error: no previous declaration for 'const target_desc* arm_linux_read_description(arm_fp_type)' [-Werror=missing-declarations]
       29 | arm_linux_read_description (arm_fp_type fp_type)
          | ^~~~~~~~~~~~~~~~~~~~~~~~~~
    /binutils-gdb/gdb/gdbserver/linux-arm-tdesc.c:49:1: error: no previous declaration for 'arm_fp_type arm_linux_get_tdesc_fp_type(const target_desc*)' [-Werror=missing-declarations]
       49 | arm_linux_get_tdesc_fp_type (const target_desc *tdesc)
          | ^~~~~~~~~~~~~~~~~~~~~~~~~~~
    
    Include linux-arm-tdesc.h in linux-arm-tdesc.c to fix it.  And because
    linux-arm-tdesc.h uses the arm_fp_type, it should include arch/arm.h as
    well.
    
    gdb/gdbserver/ChangeLog:
    
    	* linux-arm-tdesc.c: Include linux-arm-tdesc.h.
    	* linux-arm-tdesc.h: Include arch/arm.h.

Diff:
---
 gdb/gdbserver/ChangeLog         | 5 +++++
 gdb/gdbserver/linux-arm-tdesc.c | 3 +++
 gdb/gdbserver/linux-arm-tdesc.h | 2 ++
 3 files changed, 10 insertions(+)

diff --git a/gdb/gdbserver/ChangeLog b/gdb/gdbserver/ChangeLog
index 83be63c..67d39be 100644
--- a/gdb/gdbserver/ChangeLog
+++ b/gdb/gdbserver/ChangeLog
@@ -1,5 +1,10 @@
 2020-01-12  Simon Marchi  <simon.marchi@polymtl.ca>
 
+	* linux-arm-tdesc.c: Include linux-arm-tdesc.h.
+	* linux-arm-tdesc.h: Include arch/arm.h.
+
+2020-01-12  Simon Marchi  <simon.marchi@polymtl.ca>
+
 	* linux-aarch64-low.c (aarch64_write_goto_address): Make static.
 
 2020-01-12  Simon Marchi  <simon.marchi@polymtl.ca>
diff --git a/gdb/gdbserver/linux-arm-tdesc.c b/gdb/gdbserver/linux-arm-tdesc.c
index 5af1f90..2c9fac8 100644
--- a/gdb/gdbserver/linux-arm-tdesc.c
+++ b/gdb/gdbserver/linux-arm-tdesc.c
@@ -16,6 +16,9 @@
    along with this program.  If not, see <http://www.gnu.org/licenses/>.  */
 
 #include "server.h"
+
+#include "linux-arm-tdesc.h"
+
 #include "tdesc.h"
 #include "arch/arm.h"
 #include <inttypes.h>
diff --git a/gdb/gdbserver/linux-arm-tdesc.h b/gdb/gdbserver/linux-arm-tdesc.h
index bc782e1..5e8c6a3 100644
--- a/gdb/gdbserver/linux-arm-tdesc.h
+++ b/gdb/gdbserver/linux-arm-tdesc.h
@@ -18,6 +18,8 @@
 #ifndef GDBSERVER_LINUX_ARM_TDESC_H
 #define GDBSERVER_LINUX_ARM_TDESC_H
 
+#include "arch/arm.h"
+
 /* Return the Arm target description with fp registers FP_TYPE.  */
 
 const target_desc * arm_linux_read_description (arm_fp_type fp_type);