Also run foldPathConstants before mussing up SSA
authorfpizlo@apple.com <fpizlo@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 27 Apr 2018 23:43:30 +0000 (23:43 +0000)
committerfpizlo@apple.com <fpizlo@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 27 Apr 2018 23:43:30 +0000 (23:43 +0000)
https://bugs.webkit.org/show_bug.cgi?id=185069

Reviewed by Saam Barati.

This isn't needed now, but will be once I implement the phase in bug 185060.

This could be a speed-up, or a slow-down, independent of that phase. Most likely it's neutral.
Local testing seems to suggest that it's neutral. Anyway, whatever it ends up being, I want it to
be landed separately and measured separately from that phase.

It's probably nice for sanity to have this and reduceStrength run before tail duplication and
another round of reduceStrength, since that make for something that is closer to a fixpoint. But
it will increase FTL compile times. So, there's no way to guess if this change is good, bad, or
neutral. It all depends on what programs typically look like.

* b3/B3Generate.cpp:
(JSC::B3::generateToAir):

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

Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/b3/B3Generate.cpp

index 87ff7cb..8ca38cd 100644 (file)
@@ -1,3 +1,24 @@
+2018-04-26  Filip Pizlo  <fpizlo@apple.com>
+
+        Also run foldPathConstants before mussing up SSA
+        https://bugs.webkit.org/show_bug.cgi?id=185069
+
+        Reviewed by Saam Barati.
+        
+        This isn't needed now, but will be once I implement the phase in bug 185060.
+        
+        This could be a speed-up, or a slow-down, independent of that phase. Most likely it's neutral.
+        Local testing seems to suggest that it's neutral. Anyway, whatever it ends up being, I want it to
+        be landed separately and measured separately from that phase.
+        
+        It's probably nice for sanity to have this and reduceStrength run before tail duplication and
+        another round of reduceStrength, since that make for something that is closer to a fixpoint. But
+        it will increase FTL compile times. So, there's no way to guess if this change is good, bad, or
+        neutral. It all depends on what programs typically look like.
+
+        * b3/B3Generate.cpp:
+        (JSC::B3::generateToAir):
+
 2018-04-27  Ryan Haddad  <ryanhaddad@apple.com>
 
         Unreviewed, rolling out r231086.
index 675b1e2..975ba66 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2015-2017 Apple Inc. All rights reserved.
+ * Copyright (C) 2015-2018 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
@@ -87,6 +87,8 @@ void generateToAir(Procedure& procedure)
         hoistLoopInvariantValues(procedure);
         if (eliminateCommonSubexpressions(procedure))
             eliminateCommonSubexpressions(procedure);
+        foldPathConstants(procedure);
+        reduceStrength(procedure);
         inferSwitches(procedure);
         duplicateTails(procedure);
         fixSSA(procedure);