Bug 953164 - Undo debug code that wasn't part of the review, r=me

js/src/jit/Ion.cpp

@@ -1377,8 +1377,8 @@ OptimizeMIR(MIRGenerator *mir)
         // before its bounds check.
         if (!EliminateRedundantChecks(graph))
             return false;
-        AssertGraphCoherency(graph);
         IonSpewPass("Bounds Check Elimination");
+        AssertGraphCoherency(graph);
     }
 
     return true;
@@ -1396,7 +1396,7 @@ GenerateLIR(MIRGenerator *mir)
     LIRGenerator lirgen(mir, graph, *lir);
     if (!lirgen.generate())
         return nullptr;
-    //IonSpewPass("Generate LIR");
+    IonSpewPass("Generate LIR");
 
     if (mir->shouldCancel("Generate LIR"))
         return nullptr;
@@ -1419,7 +1419,7 @@ GenerateLIR(MIRGenerator *mir)
             return nullptr;
 #endif
 
-        //IonSpewPass("Allocate Registers [LSRA]", &regalloc);
+        IonSpewPass("Allocate Registers [LSRA]", &regalloc);
         break;
       }
 
@@ -1438,7 +1438,7 @@ GenerateLIR(MIRGenerator *mir)
             return nullptr;
 #endif
 
-        //IonSpewPass("Allocate Registers [Backtracking]");
+        IonSpewPass("Allocate Registers [Backtracking]");
         break;
       }
 
@@ -1453,7 +1453,7 @@ GenerateLIR(MIRGenerator *mir)
             return nullptr;
         if (!integrity.check(true))
             return nullptr;
-        //IonSpewPass("Allocate Registers [Stupid]");
+        IonSpewPass("Allocate Registers [Stupid]");
         break;
       }
 
@@ -1468,7 +1468,7 @@ GenerateLIR(MIRGenerator *mir)
     // critical edges to avoid unnecessary jumps.
     if (!UnsplitEdges(lir))
         return nullptr;
-    //IonSpewPass("Unsplit Critical Edges");
+    IonSpewPass("Unsplit Critical Edges");
     AssertBasicGraphCoherency(graph);
 
     return lir;

js/src/jit/IonAnalysis.cpp

@@ -1287,7 +1287,6 @@ jit::AssertBasicGraphCoherency(MIRGraph &graph)
 
     // Assert successor and predecessor list coherency.
     uint32_t count = 0;
-    size_t compares = 0;
     for (MBasicBlockIterator block(graph.begin()); block != graph.end(); block++) {
         count++;
 
@@ -1301,8 +1300,6 @@ jit::AssertBasicGraphCoherency(MIRGraph &graph)
 
         // Assert that use chains are valid for this instruction.
         for (MDefinitionIterator iter(*block); iter; iter++) {
-            if (iter->isCompare())
-                compares++;
             for (uint32_t i = 0, e = iter->numOperands(); i < e; i++)
                 JS_ASSERT(CheckOperandImpliesUse(*iter, iter->getOperand(i)));
         }

js/src/jit/IonBuilder.cpp

@@ -3032,13 +3032,12 @@ IonBuilder::filterTypesAtTest(MTest *test)
         return true;
 
     // Only do this optimization if the typeset does contains null or undefined.
-    if (!((removeUndefined && subject->resultTypeSet()->hasType(types::Type::UndefinedType())) ||
-        (removeNull && subject->resultTypeSet()->hasType(types::Type::NullType()))))
+    if ((!(removeUndefined && subject->resultTypeSet()->hasType(types::Type::UndefinedType())) &&
+         !(removeNull && subject->resultTypeSet()->hasType(types::Type::NullType()))))
     {
         return true;
     }
 
-    //printf("%d: replacing\n", script()->lineno());
     // Find all values on the stack that correspond to the subject
     // and replace it with a MIR with filtered TypeSet information.
     // Create the replacement MIR lazily upon first occurence.