Bug 1325344, r=jandem a=lizzard
authorHannes Verschore <hv1989@gmail.com>
Wed, 28 Dec 2016 12:56:17 +0100
changeset 357009 08e289b7046a9fc826e75455d533cbb3f250df27
parent 357008 899da67fbe51f8093ba98b3a14f3a26e63848497
child 357010 d843dc104a2e124539e43aeff6fa8a89aa1cc430
push id6710
push usercbook@mozilla.com
push dateThu, 29 Dec 2016 08:57:43 +0000
treeherdermozilla-beta@08e289b7046a [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjandem, lizzard
bugs1325344
milestone51.0
Bug 1325344, r=jandem a=lizzard
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;