Bug 1325344, r=jandem
authorHannes Verschore <hv1989@gmail.com>
Wed, 28 Dec 2016 12:56:17 +0100
changeset 327430 05f02bc144b12f72d3a40c666b4474a224a75e0d
parent 327429 32128ee674c1b88563a6fea9e2e55f42a78c2361
child 327431 29ec9f18e524058a55b5f6a70bdfeb75f6d2f6e6
push id31132
push userkwierso@gmail.com
push dateThu, 29 Dec 2016 00:09:19 +0000
treeherdermozilla-central@143bb4b9249e [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjandem
bugs1325344
milestone53.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1325344, r=jandem
js/src/jit/RangeAnalysis.cpp
--- a/js/src/jit/RangeAnalysis.cpp
+++ b/js/src/jit/RangeAnalysis.cpp
@@ -166,21 +166,18 @@ RangeAnalysis::addBetaNodes()
         BranchDirection branch_dir;
         MTest* test = block->immediateDominatorBranch(&branch_dir);
 
         if (!test || !test->getOperand(0)->isCompare())
             continue;
 
         MCompare* compare = test->getOperand(0)->toCompare();
 
-        if (compare->compareType() == MCompare::Compare_Unknown ||
-            compare->compareType() == MCompare::Compare_Bitwise)
-        {
+        if (!compare->isNumericComparison())
             continue;
-        }
 
         // TODO: support unsigned comparisons
         if (compare->compareType() == MCompare::Compare_UInt32)
             continue;
 
         MDefinition* left = compare->getOperand(0);
         MDefinition* right = compare->getOperand(1);
         double bound;