Changeset 48445 in webkit for trunk/JavaScriptCore/assembler

Timestamp:

Sep 16, 2009, 4:23:00 PM (16 years ago)

Author:

[email protected]

Message:

2009-09-16 Gabor Loki <​[email protected]>

Reviewed by Darin Adler.

Fix GCC warnings on ARM_THUMB2 platform

• assembler/ARMv7Assembler.h: (JSC::ARMThumbImmediate::countLeadingZerosPartial):
• assembler/MacroAssemblerARMv7.h: (JSC::MacroAssemblerARMv7::branchTruncateDoubleToInt32): (JSC::MacroAssemblerARMv7::moveFixedWidthEncoding):

Location:

trunk/JavaScriptCore/assembler

Files:

• ARMv7Assembler.h (modified) (1 diff)
• MacroAssemblerARMv7.h (modified) (2 diffs)

trunk/JavaScriptCore/assembler/ARMv7Assembler.h

@@ -200 +200 @@
     } PatternBytes;
 
-    ALWAYS_INLINE static int32_t countLeadingZerosPartial(uint32_t& value, int32_t& zeros, const int N)
+    ALWAYS_INLINE static void countLeadingZerosPartial(uint32_t& value, int32_t& zeros, const int N)
     {
         if (value & ~((1<<N)-1)) /* check for any of the top N bits (of 2N bits) are set */ \

trunk/JavaScriptCore/assembler/MacroAssemblerARMv7.h

@@ -533 +533 @@
     {
         ASSERT_NOT_REACHED();
+        return jump();
     }
 
@@ -1039 +1040 @@
     }
 
-    DataLabel32 moveFixedWidthEncoding(Imm32 imm, RegisterID dst)
+    void moveFixedWidthEncoding(Imm32 imm, RegisterID dst)
     {
         uint32_t value = imm.m_value;