Title: [152279] branches/dfgFourthTier/Source/_javascript_Core
Revision
152279
Author
[email protected]
Date
2013-07-01 21:33:33 -0700 (Mon, 01 Jul 2013)

Log Message

fourthTier: FTL should run LICM after AA setup
https://bugs.webkit.org/show_bug.cgi?id=118277

Reviewed by Maciej Stachowiak.
        
LICM queries alias analysis. Hence, just like GVN, it should run after
we have set up the alias analysis.

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

Modified Paths

Diff

Modified: branches/dfgFourthTier/Source/_javascript_Core/ChangeLog (152278 => 152279)


--- branches/dfgFourthTier/Source/_javascript_Core/ChangeLog	2013-07-02 04:07:28 UTC (rev 152278)
+++ branches/dfgFourthTier/Source/_javascript_Core/ChangeLog	2013-07-02 04:33:33 UTC (rev 152279)
@@ -1,5 +1,18 @@
 2013-07-01  Filip Pizlo  <[email protected]>
 
+        fourthTier: FTL should run LICM after AA setup
+        https://bugs.webkit.org/show_bug.cgi?id=118277
+
+        Reviewed by Maciej Stachowiak.
+        
+        LICM queries alias analysis. Hence, just like GVN, it should run after
+        we have set up the alias analysis.
+
+        * ftl/FTLCompile.cpp:
+        (JSC::FTL::compile):
+
+2013-07-01  Filip Pizlo  <[email protected]>
+
         fourthTier: FTL should run AA passes before GVN
         https://bugs.webkit.org/show_bug.cgi?id=118276
 

Modified: branches/dfgFourthTier/Source/_javascript_Core/ftl/FTLCompile.cpp (152278 => 152279)


--- branches/dfgFourthTier/Source/_javascript_Core/ftl/FTLCompile.cpp	2013-07-02 04:07:28 UTC (rev 152278)
+++ branches/dfgFourthTier/Source/_javascript_Core/ftl/FTLCompile.cpp	2013-07-02 04:33:33 UTC (rev 152279)
@@ -105,10 +105,10 @@
     LLVMAddConstantPropagationPass(pass);
     LLVMAddInstructionCombiningPass(pass);
     LLVMAddPromoteMemoryToRegisterPass(pass);
+    LLVMAddBasicAliasAnalysisPass(pass);
+    LLVMAddTypeBasedAliasAnalysisPass(pass);
     if (Options::enableLLVMLICM())
         LLVMAddLICMPass(pass);
-    LLVMAddBasicAliasAnalysisPass(pass);
-    LLVMAddTypeBasedAliasAnalysisPass(pass);
     LLVMAddGVNPass(pass);
     LLVMAddCFGSimplificationPass(pass);
     LLVMRunPassManager(pass, state.module);
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to