consolidate simplifyExpressions to one pass that happens before closure, where simplifyNotComps is guaranteed to be valid

2 files changed, 7 insertions, 11 deletions

diff --git a/tools/js-optimizer.js b/tools/js-optimizer.js
index 3ee1a0b6..dfc4d5dd 100644
--- a/tools/js-optimizer.js
+++ b/tools/js-optimizer.js
@@ -410,7 +410,7 @@ function removeUnneededLabelSettings(ast) {
 var USEFUL_BINARY_OPS = set('<<', '>>', '|', '&', '^');
 var COMPARE_OPS = set('<', '<=', '>', '>=', '==', '===', '!=', '!==');
 
-function simplifyExpressionsPre(ast) {
+function simplifyExpressions(ast) {
   // Simplify common expressions used to perform integer conversion operations
   // in cases where no conversion is needed.
   function simplifyIntegerConversions(ast) {
@@ -793,6 +793,7 @@ function simplifyExpressionsPre(ast) {
     joinAdditions(func);
     // simplifyZeroComp(func); TODO: investigate performance
     if (asm) asmOpts(func);
+    simplifyNotComps(func);
   });
 }
 
@@ -1158,10 +1159,6 @@ function simplifyNotComps(ast) {
   simplifyNotCompsPass = false;
 }
 
-function simplifyExpressionsPost(ast) {
-  simplifyNotComps(ast);
-}
-
 var NO_SIDE_EFFECTS = set('num', 'name');
 
 function hasSideEffects(node) { // this is 99% incomplete!
@@ -3478,10 +3475,9 @@ var passes = {
   unGlobalize: unGlobalize,
   removeAssignsToUndefined: removeAssignsToUndefined,
   //removeUnneededLabelSettings: removeUnneededLabelSettings,
-  simplifyExpressionsPre: simplifyExpressionsPre,
+  simplifyExpressions: simplifyExpressions,
   optimizeShiftsConservative: optimizeShiftsConservative,
   optimizeShiftsAggressive: optimizeShiftsAggressive,
-  simplifyExpressionsPost: simplifyExpressionsPost,
   hoistMultiples: hoistMultiples,
   loopOptimizer: loopOptimizer,
   registerize: registerize,
diff --git a/tools/test-js-optimizer-asm-pre-output.js b/tools/test-js-optimizer-asm-pre-output.js
index 8a5803d6..b31327f2 100644
--- a/tools/test-js-optimizer-asm-pre-output.js
+++ b/tools/test-js-optimizer-asm-pre-output.js
@@ -254,7 +254,7 @@ function _main($argc, $argv) {
  $j_08_i_i = 0;
  $i_09_i_i = 1;
  while (1) {
-  if (!(($j_08_i_i | 0) < 14)) {
+  if (($j_08_i_i | 0) >= 14) {
    label = 49;
    break;
   }
@@ -361,7 +361,7 @@ function _main($argc, $argv) {
  $118 = $world + 102952 | 0;
  HEAP32[$116 + 96 >> 2] = HEAP32[$118 >> 2];
  $121 = HEAP32[$118 >> 2] | 0;
- if (!(($121 | 0) == 0)) {
+ if (($121 | 0) != 0) {
   HEAP32[$121 + 92 >> 2] = $116;
  }
  HEAP32[$118 >> 2] = $116;
@@ -451,7 +451,7 @@ function _main($argc, $argv) {
    $y_sroa_0_0_insert_insert$1 = +$y_sroa_1_4_load293550;
    HEAPF32[$185 >> 2] = $y_sroa_0_0_load283451;
    HEAPF32[$185 + 4 >> 2] = $y_sroa_0_0_insert_insert$1;
-   if (!((HEAP32[$98 >> 2] & 2 | 0) == 0)) {
+   if ((HEAP32[$98 >> 2] & 2 | 0) != 0) {
     label = 65;
     break L82;
    }
@@ -466,7 +466,7 @@ function _main($argc, $argv) {
    HEAP32[$209 + 92 >> 2] = 0;
    HEAP32[$209 + 96 >> 2] = HEAP32[$118 >> 2];
    $213 = HEAP32[$118 >> 2] | 0;
-   if (!(($213 | 0) == 0)) {
+   if (($213 | 0) != 0) {
     HEAP32[$213 + 92 >> 2] = $209;
    }
    HEAP32[$118 >> 2] = $209;