Changeset 153260 in webkit

Timestamp:

Jul 24, 2013, 9:04:12 PM (12 years ago)

Author:

oliver@apple.com

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):

Location:

trunk/Source/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• ftl/FTLCompile.cpp (modified) (1 diff)

trunk/Source/JavaScriptCore/ChangeLog

@@ +1 @@
+2013-07-01  Filip Pizlo  <fpizlo@apple.com>
+
+        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  <fpizlo@apple.com>
 

trunk/Source/JavaScriptCore/ftl/FTLCompile.cpp

@@ -106 +106 @@
     LLVMAddInstructionCombiningPass(pass);
     LLVMAddPromoteMemoryToRegisterPass(pass);
+    LLVMAddBasicAliasAnalysisPass(pass);
+    LLVMAddTypeBasedAliasAnalysisPass(pass);
     if (Options::enableLLVMLICM())
         LLVMAddLICMPass(pass);
-    LLVMAddBasicAliasAnalysisPass(pass);
-    LLVMAddTypeBasedAliasAnalysisPass(pass);
     LLVMAddGVNPass(pass);
     LLVMAddCFGSimplificationPass(pass);