This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [RFC] nm header removal: Get rid of config/i386/nm-i386gnu.h header
- From: Tom Tromey <tom at tromey dot com>
- To: Thomas Schwinge <thomas at codesourcery dot com>
- Cc: <gdb-patches at sourceware dot org>, Pierre Muller <pierre dot muller at ics-cnrs dot unistra dot fr>
- Date: Thu, 14 Feb 2019 07:58:46 -0700
- Subject: Re: [RFC] nm header removal: Get rid of config/i386/nm-i386gnu.h header
- References: <000601cbe7de$3da74740$b8f5d5c0$@muller@ics-cnrs.unistra.fr> <87oc4u166a.fsf@kepler.schwinge.homeip.net> <87wom2ifyv.fsf@euler.schwinge.homeip.net>
>>>>> "Thomas" == Thomas Schwinge <thomas@codesourcery.com> writes:
Thomas> Not quite ready to remove the 'gdb/config/i386/nm-i386gnu.h' file (but
Thomas> getting there), but at least:
I have a patch to do this but I never got a response.
Maybe you could try it? I couldn't readily find it in the archives so I
have appended it again.
Tom
commit 86d2fb04c1890b381a24d3508087d11cc88388c8
Author: Tom Tromey <tom@tromey.com>
Date: Mon Dec 24 14:49:27 2018 -0700
Attempt to fix Hurd build
gdb/ChangeLog
2018-12-24 Tom Tromey <tom@tromey.com>
* gnu-nat.c (THREAD_STATE_FLAVOR, THREAD_STATE_SIZE)
(THREAD_STATE_SET_TRACED, THREAD_STATE_CLEAR_TRACED): Move
definitions from nm-i386gnu.h.
* configure.nat: Don't reference nm-i386gnu.h.
* config/i386/nm-i386gnu.h: Remove.
diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 21731be98db..bc408fe7481 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,11 @@
+2018-12-24 Tom Tromey <tom@tromey.com>
+
+ * gnu-nat.c (THREAD_STATE_FLAVOR, THREAD_STATE_SIZE)
+ (THREAD_STATE_SET_TRACED, THREAD_STATE_CLEAR_TRACED): Move
+ definitions from nm-i386gnu.h.
+ * configure.nat: Don't reference nm-i386gnu.h.
+ * config/i386/nm-i386gnu.h: Remove.
+
2018-12-18 Tom Tromey <tom@tromey.com>
* dwarf2read.c (dwarf2_find_containing_comp_unit): Don't take
diff --git a/gdb/config/i386/nm-i386gnu.h b/gdb/config/i386/nm-i386gnu.h
deleted file mode 100644
index 4b2ca907fda..00000000000
--- a/gdb/config/i386/nm-i386gnu.h
+++ /dev/null
@@ -1,37 +0,0 @@
-/* Native-dependent definitions for Intel 386 running the GNU Hurd
- Copyright (C) 1994-2018 Free Software Foundation, Inc.
-
- This file is part of GDB.
-
- This program is free software; you can redistribute it and/or modify
- it under the terms of the GNU General Public License as published by
- the Free Software Foundation; either version 3 of the License, or
- (at your option) any later version.
-
- This program is distributed in the hope that it will be useful,
- but WITHOUT ANY WARRANTY; without even the implied warranty of
- MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- GNU General Public License for more details.
-
- You should have received a copy of the GNU General Public License
- along with this program. If not, see <http://www.gnu.org/licenses/>. */
-
-#ifndef NM_I386GNU_H
-#define NM_I386GNU_H
-
-#include <unistd.h>
-#include <mach.h>
-#include <mach/exception.h>
-#include "regcache.h"
-
-extern char *gnu_target_pid_to_str (int pid);
-
-/* Thread flavors used in re-setting the T bit. */
-#define THREAD_STATE_FLAVOR i386_REGS_SEGS_STATE
-#define THREAD_STATE_SIZE i386_THREAD_STATE_COUNT
-#define THREAD_STATE_SET_TRACED(state) \
- ((struct i386_thread_state *) (state))->efl |= 0x100
-#define THREAD_STATE_CLEAR_TRACED(state) \
- ((((struct i386_thread_state *) (state))->efl &= ~0x100), 1)
-
-#endif /* nm-i386gnu.h */
diff --git a/gdb/configure.nat b/gdb/configure.nat
index 200b7169243..aa6f9c5c4e3 100644
--- a/gdb/configure.nat
+++ b/gdb/configure.nat
@@ -218,7 +218,6 @@ case ${gdb_host} in
msg_U.o exc_request_U.o exc_request_S.o'
HAVE_NATIVE_GCORE_HOST=1
- NAT_FILE='nm-i386gnu.h'
MH_CFLAGS='-D_GNU_SOURCE'
XM_CLIBS='-lshouldbeinlibc'
diff --git a/gdb/gnu-nat.c b/gdb/gnu-nat.c
index a886c2b80ce..612815f8ae8 100644
--- a/gdb/gnu-nat.c
+++ b/gdb/gnu-nat.c
@@ -81,6 +81,14 @@ extern "C"
#include "msg_U.h"
}
+/* Thread flavors used in re-setting the T bit. */
+#define THREAD_STATE_FLAVOR i386_REGS_SEGS_STATE
+#define THREAD_STATE_SIZE i386_THREAD_STATE_COUNT
+#define THREAD_STATE_SET_TRACED(state) \
+ ((struct i386_thread_state *) (state))->efl |= 0x100
+#define THREAD_STATE_CLEAR_TRACED(state) \
+ ((((struct i386_thread_state *) (state))->efl &= ~0x100), 1)
+
static process_t proc_server = MACH_PORT_NULL;
/* If we've sent a proc_wait_request to the proc server, the pid of the