Changeset 61745 in webkit for trunk/JavaScriptCore/assembler

Timestamp:

Jun 24, 2010, 12:48:37 AM (15 years ago)

Author:

[email protected]

Message:

Fix the length of instruction stream controlled by constant pool
​https://bugs.webkit.org/show_bug.cgi?id=40293

Reviewed by Gavin Barraclough.

The initial/maximum length of instruction stream (m_maxDistance) should
be set when the first constant arrives to the constant pool. Otherwise
the constant pool could be placed into an uninterrupted sequence.

• assembler/AssemblerBufferWithConstantPool.h:

(JSC::):

File:

• trunk/JavaScriptCore/assembler/AssemblerBufferWithConstantPool.h (modified) (2 diffs)

trunk/JavaScriptCore/assembler/AssemblerBufferWithConstantPool.h

@@ -191 +191 @@
     void putIntWithConstantInt(uint32_t insn, uint32_t constant, bool isReusable = false)
     {
+        if (!m_numConsts)
+            m_maxDistance = maxPoolSize;
         flushIfNoSpaceFor(4, 4);
 
@@ -280 +282 @@
         m_loadOffsets.clear();
         m_numConsts = 0;
-        m_maxDistance = maxPoolSize;
     }