Bug 1325344, r=jandem a=lizzard
authorHannes Verschore <hv1989@gmail.com>
Wed, 28 Dec 2016 12:56:17 +0100
changeset 353221 9ec81934225f1db91e1a69d828536e0d00a52a70
parent 353220 e3023975fdec14510f7c7e6117f05f967add3fbf
child 353222 5ae4b9494db7c437b5c3a114431aab12b93b8d03
push id6795
push userjlund@mozilla.com
push dateMon, 23 Jan 2017 14:19:46 +0000
treeherdermozilla-esr52@76101b503191 [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;