Bug 923659 - IonMonkey: Fix an accidentally dropped "break". r=nbp
authorDan Gohman <sunfish@google.com>
Tue, 15 Oct 2013 20:49:43 -0700
changeset 164679 4a67b5951c2313f6df4e42044417e68dee452342
parent 164678 e3c8bbbfec049f67fc3e8611d96f106247a8d2f8
child 164680 22e6f21251c929738ed884970d4c011c032dff18
push id3066
push userakeybl@mozilla.com
push dateMon, 09 Dec 2013 19:58:46 +0000
treeherdermozilla-beta@a31a0dce83aa [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());