Optionally use 31-bits SMI value for 64-bit system

include/v8.h

Index: include/v8.h
diff --git a/include/v8.h b/include/v8.h
index 36077402a287046e84cf5f4dc8c1d6bd120a3ab3..01b9124cfacbdda198b47f03be45237739794ece 100644
--- a/include/v8.h
+++ b/include/v8.h
@@ -5347,6 +5347,7 @@ template <> struct SmiTagging<4> {
   }
 };
 
+#if !V8_USE_31_BITS_SMI_VALUE

[danno, 2013/07/29 13:02:23]

I also see no reason to put the SmiTagging<8> definition in a #ifdef

Also, reverse the sense of the #ifdef


#if V8_USE_31_BIT_SMI_VALUE
typedef SmiTagging<4> PlatformSmiTagging;
#else
typedef SmiTagging<kApiPointerSize> PlatformSmiTagging;
#endif

[haitao.feng, 2013/07/30 08:56:43]

Done.

 // Smi constants for 64-bit systems.
 template <> struct SmiTagging<8> {
   static const int kSmiShiftSize = 31;
@@ -5366,6 +5367,10 @@ template <> struct SmiTagging<8> {
 };
 
 typedef SmiTagging<kApiPointerSize> PlatformSmiTagging;
+#else
+typedef SmiTagging<4> PlatformSmiTagging;
+#endif
+
 const int kSmiShiftSize = PlatformSmiTagging::kSmiShiftSize;
 const int kSmiValueSize = PlatformSmiTagging::kSmiValueSize;