From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 2865 invoked by alias); 13 Sep 2003 17:02:58 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 2854 invoked from network); 13 Sep 2003 17:02:57 -0000 Received: from unknown (HELO redhat.com) (24.131.133.249) by sources.redhat.com with SMTP; 13 Sep 2003 17:02:57 -0000 Received: by redhat.com (Postfix, from userid 201) id 38ADD32A822; Sat, 13 Sep 2003 13:02:54 -0400 (EDT) Date: Sat, 13 Sep 2003 17:02:00 -0000 From: Christopher Faylor To: gdb-patches@sources.redhat.com Subject: [PATCH] Make SSE registers unconditionally available on cygwin Message-ID: <20030913170254.GA21503@redhat.com> Mail-Followup-To: gdb-patches@sources.redhat.com References: <20030913022759.GA2225@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20030913022759.GA2225@redhat.com> User-Agent: Mutt/1.4.1i X-SW-Source: 2003-09/txt/msg00291.txt.bz2 I've made the below change sort of at the request of Mark Kettenis but it also occurred to me that trying to accommodate older versions of w32api headers which did not define extended registers was pointless since gdb crashes in some situations if the registers aren't available. Rather than investigate what needs to be done to accommodate ancient versions of cygwin, I opted to just produce a compile error for the situation where extended registers are not available. Updating header files shouldn't be that onerous a task if this problem hits someone. cgf 2003-09-13 Christopher Faylor * win32-nat.c: Just rely on CONTEXT_EXTENDED_REGISTER being defined for SSE registers since gdb will not operate correctly without this. Restore include file ordering munged in previous change. * config/i386/tm-cygwin.h: Remove HAVE_SSE_REGS define. Index: win32-nat.c =================================================================== RCS file: /cvs/uberbaum/gdb/win32-nat.c,v retrieving revision 1.78 diff -u -p -r1.78 win32-nat.c --- win32-nat.c 13 Sep 2003 02:26:42 -0000 1.78 +++ win32-nat.c 13 Sep 2003 16:05:24 -0000 @@ -26,15 +26,7 @@ /* We assume we're being built with and will be used for cygwin. */ -#include -#include -#include -#include -#include -#include -#include #include "defs.h" -#include "tm.h" /* required for SSE registers */ #include "frame.h" /* required by inferior.h */ #include "inferior.h" #include "target.h" @@ -44,6 +36,13 @@ #include "regcache.h" #include "top.h" #include "i386-tdep.h" +#include +#include +#include +#include +#include +#include +#include #include "buildsym.h" #include "symfile.h" @@ -70,12 +69,8 @@ enum #include #include -#ifdef HAVE_SSE_REGS #define CONTEXT_DEBUGGER_DR CONTEXT_DEBUGGER | CONTEXT_DEBUG_REGISTERS \ | CONTEXT_EXTENDED_REGISTERS -#else -#define CONTEXT_DEBUGGER_DR CONTEXT_DEBUGGER | CONTEXT_DEBUG_REGISTERS -#endif static unsigned dr[8]; static int debug_registers_changed = 0; Index: config/i386/tm-cygwin.h =================================================================== RCS file: /cvs/uberbaum/gdb/config/i386/tm-cygwin.h,v retrieving revision 1.15 diff -u -p -r1.15 tm-cygwin.h --- config/i386/tm-cygwin.h 13 Sep 2003 02:26:42 -0000 1.15 +++ config/i386/tm-cygwin.h 13 Sep 2003 16:05:24 -0000 @@ -19,13 +19,6 @@ Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. */ -/* Use SSE registers if winnt.h contains information about them. */ -#ifdef CONTEXT_EXTENDED_REGISTERS -#define HAVE_SSE_REGS -#else -#undef HAVE_SSE_REGS -#endif /* CONTEXT_EXTENDED_REGISTERS */ - #include "i386/tm-i386.h" #define ATTACH_NO_WAIT