Bug 923659 - IonMonkey: Fix an accidentally dropped "break". r=nbp
authorDan Gohman <sunfish@google.com>
Tue, 15 Oct 2013 20:49:43 -0700
changeset 165631 4a67b5951c2313f6df4e42044417e68dee452342
parent 165630 e3c8bbbfec049f67fc3e8611d96f106247a8d2f8
child 165632 22e6f21251c929738ed884970d4c011c032dff18
push id428
push userbbajaj@mozilla.com
push dateTue, 28 Jan 2014 00:16:25 +0000
treeherdermozilla-release@cd72a7ff3a75 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersnbp
bugs923659
milestone27.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 923659 - IonMonkey: Fix an accidentally dropped "break". r=nbp
js/src/jit/RangeAnalysis.cpp
--- a/js/src/jit/RangeAnalysis.cpp
+++ b/js/src/jit/RangeAnalysis.cpp
@@ -225,16 +225,17 @@ RangeAnalysis::addBetaNodes()
                 int32_t intbound;
                 if (DoubleIsInt32(bound, &intbound) && SafeAdd(intbound, 1, &intbound))
                     bound = intbound;
             }
             comp.set(floor(bound), Range::NoInt32UpperBound, true, exponent);
             break;
           case JSOP_EQ:
             comp.set(floor(bound), ceil(bound), true, exponent);
+            break;
           default:
             continue; // well, for neq we could have
                       // [-\inf, bound-1] U [bound+1, \inf] but we only use contiguous ranges.
         }
 
         if (IonSpewEnabled(IonSpew_Range)) {
             IonSpewHeader(IonSpew_Range);
             fprintf(IonSpewFile, "Adding beta node for %d with range ", val->id());