This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[patch windows]: Don't define _X86_ for 64-bit targets


Hi,

As I noticed is in cygming.h the define _X86_ predefined. But for
64-bit it shouldn't. So I adjusted the definition so, that it is
defined just for 32-bit case.

ChangeLog

2009-07-12  Kai Tietz  <kai.tietz@onevision.com>

	* config/i386/cygming.h (TARGET_OS_CPP_BUILTINS): Define _X86_
	just for 32-bit case.

I'll apply this patch in a couple of hours for trunk and 4.4 branch,
if there aren't any objections.

Cheers,
Kai

-- 
|  (\_/) This is Bunny. Copy and paste
| (='.'=) Bunny into your signature to help
| (")_(") him gain world domination
Index: gcc/gcc/config/i386/cygming.h
===================================================================
--- gcc.orig/gcc/config/i386/cygming.h	2009-03-29 16:29:54.000000000 +0200
+++ gcc/gcc/config/i386/cygming.h	2009-07-12 10:56:23.503073500 +0200
@@ -73,7 +73,8 @@
 #define TARGET_OS_CPP_BUILTINS()					\
   do									\
     {									\
-	builtin_define ("_X86_=1");					\
+	if (!TARGET_64BIT)						\
+	  builtin_define ("_X86_=1");					\
 	builtin_assert ("system=winnt");				\
 	builtin_define ("__stdcall=__attribute__((__stdcall__))");	\
 	builtin_define ("__fastcall=__attribute__((__fastcall__))");	\

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]