[v3,2/3] Define _KERNTYPES in arm-nbsd-nat.c

Message ID 20200124143401.177048-1-cbiesinger@chromium.org
State New, archived
Headers

Commit Message

Christian Biesinger Jan. 24, 2020, 2:34 p.m. UTC
  From: Christian Biesinger <cbiesinger@google.com>

Fixes the below compile error on ARM NetBSD 9.0_RC1 (the only version I
tested).  types.h does not define register_t by default.

We already use this define elsewhere, notably in bsd-kvm.c.

In file included from ../../gdb/arm-nbsd-nat.c:28:
/usr/include/machine/frame.h:54:2: error: unknown type name 'register_t'; did you mean '__register_t'?
        register_t tf_spsr;
        ^
/usr/include/machine/types.h:77:14: note: '__register_t' declared here
typedef int             __register_t;
                        ^

There are other compile errors that this does not fix.

gdb/ChangeLog:

2020-01-24  Christian Biesinger  <cbiesinger@google.com>

	* arm-nbsd-nat.c: Define _KERNTYPES to get the declaration of
	register_t.

Change-Id: I82c21d38189ee59ea0af2538ba84b771d268722e
---
 gdb/arm-nbsd-nat.c | 2 ++
 1 file changed, 2 insertions(+)
  

Comments

Tom Tromey Jan. 24, 2020, 3:03 p.m. UTC | #1
>>>>> ">" == cbiesinger  <cbiesinger@chromium.org> writes:

>> 2020-01-24  Christian Biesinger  <cbiesinger@google.com>

>> 	* arm-nbsd-nat.c: Define _KERNTYPES to get the declaration of
>> 	register_t.

Thanks, this is ok.

Tom
  

Patch

diff --git a/gdb/arm-nbsd-nat.c b/gdb/arm-nbsd-nat.c
index 00f919194b..33ae790063 100644
--- a/gdb/arm-nbsd-nat.c
+++ b/gdb/arm-nbsd-nat.c
@@ -17,6 +17,8 @@ 
    You should have received a copy of the GNU General Public License
    along with this program.  If not, see <http://www.gnu.org/licenses/>.  */
 
+/* We define this to get types like register_t.  */
+#define _KERNTYPES
 #include "defs.h"
 #include "gdbcore.h"
 #include "inferior.h"