FTL should use the new version of LLVM MCJIT memory manager APIs that take a SectionName
authorfpizlo@apple.com <fpizlo@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 2 Oct 2013 02:22:56 +0000 (02:22 +0000)
committerfpizlo@apple.com <fpizlo@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 2 Oct 2013 02:22:56 +0000 (02:22 +0000)
https://bugs.webkit.org/show_bug.cgi?id=122193

Source/JavaScriptCore:

Reviewed by Geoffrey Garen.

Update our usage of the LLVM C API since the API is about to change.

* ftl/FTLCompile.cpp:
(JSC::FTL::mmAllocateCodeSection):
(JSC::FTL::mmAllocateDataSection):

WebKitLibraries:

Reviewed by Geoffrey Garen.

* LLVMIncludesMountainLion.tar.bz2:
* LLVMLibrariesMountainLion.tar.bz2:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@156758 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/ftl/FTLCompile.cpp
WebKitLibraries/ChangeLog
WebKitLibraries/LLVMIncludesMountainLion.tar.bz2
WebKitLibraries/LLVMLibrariesMountainLion.tar.bz2

index 31471c9..227037f 100644 (file)
@@ -1,5 +1,18 @@
 2013-10-01  Filip Pizlo  <fpizlo@apple.com>
 
+        FTL should use the new version of LLVM MCJIT memory manager APIs that take a SectionName
+        https://bugs.webkit.org/show_bug.cgi?id=122193
+
+        Reviewed by Geoffrey Garen.
+        
+        Update our usage of the LLVM C API since the API is about to change.
+
+        * ftl/FTLCompile.cpp:
+        (JSC::FTL::mmAllocateCodeSection):
+        (JSC::FTL::mmAllocateDataSection):
+
+2013-10-01  Filip Pizlo  <fpizlo@apple.com>
+
         REGRESSION(156464): 50% regression on SunSpider/string-fasta
         https://bugs.webkit.org/show_bug.cgi?id=122202
 
index 4649477..d13f9b7 100644 (file)
@@ -42,9 +42,11 @@ namespace JSC { namespace FTL {
 using namespace DFG;
 
 static uint8_t* mmAllocateCodeSection(
-    void* opaqueState, uintptr_t size, unsigned alignment, unsigned sectionID)
+    void* opaqueState, uintptr_t size, unsigned alignment, unsigned sectionID,
+    const char* sectionName)
 {
     UNUSED_PARAM(sectionID);
+    UNUSED_PARAM(sectionName);
     
     State& state = *static_cast<State*>(opaqueState);
     
@@ -61,13 +63,13 @@ static uint8_t* mmAllocateCodeSection(
 
 static uint8_t* mmAllocateDataSection(
     void* opaqueState, uintptr_t size, unsigned alignment, unsigned sectionID,
-    LLVMBool isReadOnly)
+    const char* sectionName, LLVMBool isReadOnly)
 {
     // FIXME: fourthTier: FTL memory allocator should be able to allocate data
     // sections in non-executable memory.
     // https://bugs.webkit.org/show_bug.cgi?id=116189
     UNUSED_PARAM(isReadOnly);
-    return mmAllocateCodeSection(opaqueState, size, alignment, sectionID);
+    return mmAllocateCodeSection(opaqueState, size, alignment, sectionID, sectionName);
 }
 
 static LLVMBool mmApplyPermissions(void*, char**)
index 4aa570a..0faf44c 100644 (file)
@@ -1,3 +1,13 @@
+2013-10-01  Filip Pizlo  <fpizlo@apple.com>
+
+        FTL should use the new version of LLVM MCJIT memory manager APIs that take a SectionName
+        https://bugs.webkit.org/show_bug.cgi?id=122193
+
+        Reviewed by Geoffrey Garen.
+
+        * LLVMIncludesMountainLion.tar.bz2:
+        * LLVMLibrariesMountainLion.tar.bz2:
+
 2013-10-01  Brent Fulgham  <bfulgham@apple.com>
 
         [Win] Implement the media controls in JavaScript.
index f246564..ac162c0 100644 (file)
Binary files a/WebKitLibraries/LLVMIncludesMountainLion.tar.bz2 and b/WebKitLibraries/LLVMIncludesMountainLion.tar.bz2 differ
index 9af3d3b..85da656 100644 (file)
Binary files a/WebKitLibraries/LLVMLibrariesMountainLion.tar.bz2 and b/WebKitLibraries/LLVMLibrariesMountainLion.tar.bz2 differ