diff mbox

PATCH: Use long long for 64bit int in config/i386/64/sfp-machine.h

Message ID 20110728194547.GA16325@intel.com
State New
Headers show

Commit Message

H.J. Lu July 28, 2011, 7:45 p.m. UTC 
Hi Ian,

For 64bit x86 targets, long is 32bit for x32 and win64.  But long long
is always 64bit.  This patch removes _WIN64 check.  OK for trunk?

Thanks.


H.J.
---
2010-07-28  H.J. Lu  <hongjiu.lu@intel.com>

	* config/i386/64/sfp-machine.h (_FP_W_TYPE): Always use _WIN64
	version.
	(_FP_WS_TYPE): Likewise.
	(_FP_I_TYPE): Likewise.

Comments

Richard Henderson July 28, 2011, 8:06 p.m. UTC | #1 
On 07/28/2011 12:45 PM, H.J. Lu wrote:
> 	* config/i386/64/sfp-machine.h (_FP_W_TYPE): Always use _WIN64
> 	version.
> 	(_FP_WS_TYPE): Likewise.
> 	(_FP_I_TYPE): Likewise.

Ok.


r~
NightStrike July 29, 2011, 7:57 p.m. UTC | #2 
On Thu, Jul 28, 2011 at 3:45 PM, H.J. Lu <hongjiu.lu@intel.com> wrote:
> Hi Ian,
>
> For 64bit x86 targets, long is 32bit for x32 and win64.  But long long
> is always 64bit.  This patch removes _WIN64 check.  OK for trunk?

Isn't that what int64_t is for?
diff mbox

Patch

diff --git a/libgcc/config/i386/64/sfp-machine.h b/libgcc/config/i386/64/sfp-machine.h
index 5adf6db..5debf5a 100644
--- a/libgcc/config/i386/64/sfp-machine.h
+++ b/libgcc/config/i386/64/sfp-machine.h
@@ -1,14 +1,8 @@ 
 #define _FP_W_TYPE_SIZE		64
 
-#ifdef _WIN64
- #define _FP_W_TYPE		unsigned long long
- #define _FP_WS_TYPE		signed long long
- #define _FP_I_TYPE		long long
-#else
- #define _FP_W_TYPE		unsigned long
- #define _FP_WS_TYPE		signed long
- #define _FP_I_TYPE		long
-#endif
+#define _FP_W_TYPE		unsigned long long
+#define _FP_WS_TYPE		signed long long
+#define _FP_I_TYPE		long long
 
 typedef int TItype __attribute__ ((mode (TI)));
 typedef unsigned int UTItype __attribute__ ((mode (TI)));