Title: [150470] branches/dfgFourthTier/Source/_javascript_Core
Revision
150470
Author
[email protected]
Date
2013-05-21 12:43:30 -0700 (Tue, 21 May 2013)

Log Message

fourthTier: FTL shouldn't use FastISel and Small code model should be turned off for now
https://bugs.webkit.org/show_bug.cgi?id=115998

Reviewed by Oliver Hunt.
        
This switches off FastISel and makes it possible to turn off Small code model.

* ftl/FTLCompile.cpp:
(JSC::FTL::compile):
* runtime/Options.h:
(JSC):

Modified Paths

Diff

Modified: branches/dfgFourthTier/Source/_javascript_Core/ChangeLog (150469 => 150470)


--- branches/dfgFourthTier/Source/_javascript_Core/ChangeLog	2013-05-21 19:34:31 UTC (rev 150469)
+++ branches/dfgFourthTier/Source/_javascript_Core/ChangeLog	2013-05-21 19:43:30 UTC (rev 150470)
@@ -1,3 +1,17 @@
+2013-05-12  Filip Pizlo  <[email protected]>
+
+        fourthTier: FTL shouldn't use FastISel and Small code model should be turned off for now
+        https://bugs.webkit.org/show_bug.cgi?id=115998
+
+        Reviewed by Oliver Hunt.
+        
+        This switches off FastISel and makes it possible to turn off Small code model.
+
+        * ftl/FTLCompile.cpp:
+        (JSC::FTL::compile):
+        * runtime/Options.h:
+        (JSC):
+
 2013-05-21  Filip Pizlo  <[email protected]>
 
         fourthTier: should use ConcurrentJITLock[er] directly and not through typedef
@@ -955,7 +969,6 @@
         * runtime/Options.h:
         (JSC):
 
->>>>>>> .r149942
 2013-05-10  Filip Pizlo  <[email protected]>
 
         fourthTier: FTL should support CompareStrictEqConstant

Modified: branches/dfgFourthTier/Source/_javascript_Core/ftl/FTLCompile.cpp (150469 => 150470)


--- branches/dfgFourthTier/Source/_javascript_Core/ftl/FTLCompile.cpp	2013-05-21 19:34:31 UTC (rev 150469)
+++ branches/dfgFourthTier/Source/_javascript_Core/ftl/FTLCompile.cpp	2013-05-21 19:43:30 UTC (rev 150470)
@@ -47,7 +47,8 @@
     LLVMInitializeMCJITCompilerOptions(&options, sizeof(options));
     options.OptLevel = Options::llvmOptimizationLevel();
     options.NoFramePointerElim = true;
-    options.CodeModel = LLVMCodeModelSmall;
+    if (Options::useLLVMSmallCodeModel())
+        options.CodeModel = LLVMCodeModelSmall;
     options.EnableFastISel = Options::enableLLVMFastISel();
     
     if (LLVMCreateMCJITCompilerForModule(&state.engine, state.module, &options, sizeof(options), &error)) {

Modified: branches/dfgFourthTier/Source/_javascript_Core/runtime/Options.h (150469 => 150470)


--- branches/dfgFourthTier/Source/_javascript_Core/runtime/Options.h	2013-05-21 19:34:31 UTC (rev 150469)
+++ branches/dfgFourthTier/Source/_javascript_Core/runtime/Options.h	2013-05-21 19:43:30 UTC (rev 150470)
@@ -87,7 +87,8 @@
     v(bool, useExperimentalFTL, false) \
     v(bool, useFTLTBAA, true) \
     v(bool, enableLLVMLICM, true) \
-    v(bool, enableLLVMFastISel, true) \
+    v(bool, enableLLVMFastISel, false) \
+    v(bool, useLLVMSmallCodeModel, false) \
     v(unsigned, llvmOptimizationLevel, 2) \
     \
     v(bool, enableConcurrentJIT, true) \
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to