Changeset 39083 in webkit for trunk/JavaScriptCore/wtf/FastMalloc.cpp

Timestamp:

Dec 7, 2008, 3:55:04 PM (16 years ago)

Author:

[email protected]

Message:

<rdar://problem/6309878> Need more granular control over allocation of executable memory (21783)
<​https://bugs.webkit.org/show_bug.cgi?id=21783>

Reviewed by Cameron Zwarich and Sam Weinig

Add a new allocator for use by the JIT that provides executable pages, so
we can get rid of the current hack that makes the entire heap executable.

1-2% progression on SunSpider-v8, 1% on SunSpider. Reduces memory usage as well!

File:

• trunk/JavaScriptCore/wtf/FastMalloc.cpp (modified) (2 diffs)

trunk/JavaScriptCore/wtf/FastMalloc.cpp

@@ -240 +240 @@
 }
 
-#if HAVE(VIRTUALALLOC)
-void* fastMallocExecutable(size_t n)
-{
-    return VirtualAlloc(0, n, MEM_COMMIT | MEM_RESERVE, PAGE_EXECUTE_READWRITE);
-}
-
-void fastFreeExecutable(void* p)
-{ 
-    VirtualFree(p, 0, MEM_RELEASE); 
-}
-#else
-void* fastMallocExecutable(size_t n)
-{
-    return fastMalloc(n);
-}
-
-void fastFreeExecutable(void* p)
-{ 
-    fastFree(p);
-}
-#endif
-
 } // namespace WTF
 
@@ -3405 +3383 @@
     return old_ptr;
   }
-}
-
-void* fastMallocExecutable(size_t n)
-{
-    return malloc<false>(n);
-}
-
-void fastFreeExecutable(void* p)
-{ 
-    free(p);
 }