Changeset 37283 in webkit for trunk/JavaScriptCore/VM

Timestamp:

Oct 3, 2008, 10:47:38 PM (17 years ago)

Author:

[email protected]

Message:

2008-10-03 Cameron Zwarich <​[email protected]>

Reviewed by Maciej Stachowiak.

Bug 21356: The size of the RegisterFile differs depending on 32-bit / 64-bit and Debug / Release
<​https://bugs.webkit.org/show_bug.cgi?id=21356>

The RegisterFile decreases in size (measured in terms of numbers of
Registers) as the size of a Register increases. This causes

js1_5/Regress/regress-159334.js

to fail in 64-bit debug builds. This fix makes the RegisterFile on all
platforms the same size that it is in 32-bit Release builds.

• VM/RegisterFile.h: (JSC::RegisterFile::RegisterFile):

File:

• trunk/JavaScriptCore/VM/RegisterFile.h (modified) (1 diff)

trunk/JavaScriptCore/VM/RegisterFile.h

@@ -107 +107 @@
         enum { ArgumentsRegister = 0 };
 
-        enum { DefaultCapacity = 2 * 1024 * 1024 / sizeof(Register) };
-        enum { DefaultMaxGlobals = 8 * 1024 };
+        static const size_t defaultCapacity = 524288;
+        static const size_t defaultMaxGlobals = 8192;
 
-        RegisterFile(size_t capacity = DefaultCapacity, size_t maxGlobals = DefaultMaxGlobals)
+        RegisterFile(size_t capacity = defaultCapacity, size_t maxGlobals = defaultMaxGlobals)
             : m_numGlobals(0)
             , m_maxGlobals(maxGlobals)