Changeset 94922 in webkit for trunk/Source/JavaScriptCore/assembler/ARMAssembler.cpp

Timestamp:

Sep 11, 2011, 12:08:25 AM (14 years ago)

Author:

Csaba Osztrogonác

Message:

ARM and MIPS assemblers still refer to executable pools.
​https://bugs.webkit.org/show_bug.cgi?id=67903

Patch by Filip Pizlo <​[email protected]> on 2011-09-11
Reviewed by Csaba Osztrogonác.

• assembler/ARMAssembler.cpp:

(JSC::ARMAssembler::executableCopy):

• assembler/ARMAssembler.h:
• assembler/AssemblerBufferWithConstantPool.h:
• assembler/MIPSAssembler.h:

(JSC::MIPSAssembler::executableCopy):

File:

• trunk/Source/JavaScriptCore/assembler/ARMAssembler.cpp (modified) (3 diffs)

trunk/Source/JavaScriptCore/assembler/ARMAssembler.cpp

@@ -344 +344 @@
 }
 
-void* ARMAssembler::executableCopy(JSGlobalData& globalData, ExecutablePool* allocator)
+PassRefPtr<ExecutableMemoryHandle> ARMAssembler::executableCopy(JSGlobalData& globalData)
 {
     // 64-bit alignment is required for next constant pool and JIT code as well
@@ -351 +351 @@
         bkpt(0);
 
-    char* data = reinterpret_cast<char*>(m_buffer.executableCopy(globalData, allocator));
+    RefPtr<ExecutableMemoryHandle> result = m_buffer.executableCopy(globalData);
+    char* data = reinterpret_cast<char*>(result->start());
 
     for (Jumps::Iterator iter = m_jumps.begin(); iter != m_jumps.end(); ++iter) {
@@ -371 +372 @@
     }
 
-    return data;
+    return result;
 }