Changeset 31115 in webkit

Timestamp:

Mar 17, 2008, 6:54:48 PM (17 years ago)

Author:

ddkilzer@apple.com

Message:

Don't define PLATFORM(MIDDLE_ENDIAN) on little endian ARM.

Reviewed by Darin.

See <​http://bugs.webkit.org/show_bug.cgi?id=15416#c13>.

• wtf/Platform.h: Added check for !defined(ARMEL) when defining PLATFORM(MIDDLE_ENDIAN).

Location:

trunk/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• wtf/Platform.h (modified) (2 diffs)

trunk/JavaScriptCore/ChangeLog

@@ +1 @@
+2008-03-17  David Kilzer  <ddkilzer@apple.com>
+
+        Don't define PLATFORM(MIDDLE_ENDIAN) on little endian ARM.
+
+        Reviewed by Darin.
+
+        See <http://bugs.webkit.org/show_bug.cgi?id=15416#c13>.
+
+        * wtf/Platform.h: Added check for !defined(__ARMEL__) when defining
+        PLATFORM(MIDDLE_ENDIAN).
+
 2008-03-17  Oliver Hunt  <oliver@apple.com>
 

trunk/JavaScriptCore/wtf/Platform.h

@@ -143 +143 @@
 #endif
 
+/* PLATFORM(ARM) */
 #if   defined(arm) \
    || defined(__arm__)
@@ -148 +149 @@
 #if defined(__ARMEB__)
 #define WTF_PLATFORM_BIG_ENDIAN 1
-#elif !defined(__ARM_EABI__) && !defined(__ARMEB__)
+#elif !defined(__ARM_EABI__) && !defined(__ARMEB__) && !defined(__ARMEL__)
 #define WTF_PLATFORM_MIDDLE_ENDIAN 1
 #endif