Bug 1325344, r=jandem a=lizzard
authorHannes Verschore <hv1989@gmail.com>
Wed, 28 Dec 2016 12:56:17 +0100
changeset 349869 9ec81934225f1db91e1a69d828536e0d00a52a70
parent 349868 e3023975fdec14510f7c7e6117f05f967add3fbf
child 349870 5ae4b9494db7c437b5c3a114431aab12b93b8d03
push id10488
push usercbook@mozilla.com
push dateThu, 29 Dec 2016 08:40:43 +0000
treeherdermozilla-aurora@3e93be52e058 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjandem, lizzard
bugs1325344
milestone52.0a2
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;