Bug 824005: Fix for v8-richards regression introduced in bug 813784, r=nbp
authorHannes Verschore <hv1989@gmail.com>
Sat, 22 Dec 2012 17:09:32 +0100
changeset 116896 b9807d681190145ce7a0ee12439d9d909596f31a
parent 116895 ec5b0408864fe5f741a39676c704f45690b7f721
child 116897 1911ef1478a37f9078a37376594e7402969f86a1
push id24076
push userryanvm@gmail.com
push dateSun, 23 Dec 2012 20:50:19 +0000
treeherdermozilla-central@4f74d77d6d8b [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersnbp
bugs824005, 813784
milestone20.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 824005: Fix for v8-richards regression introduced in bug 813784, r=nbp
js/src/jsanalyze.cpp
--- a/js/src/jsanalyze.cpp
+++ b/js/src/jsanalyze.cpp
@@ -588,19 +588,20 @@ ScriptAnalysis::analyzeBytecode(JSContex
           case JSOP_INT32:
           case JSOP_HOLE:
           case JSOP_LOOPHEAD:
           case JSOP_LOOPENTRY:
           case JSOP_NOTEARG:
             break;
 
           default:
-            if (!(js_CodeSpec[op].format & JOF_DECOMPOSE))
+            if (!(js_CodeSpec[op].format & JOF_DECOMPOSE)) {
                 isJaegerCompileable = false;
                 isJaegerInlineable = isIonInlineable = false;
+            }
             break;
         }
 
         bool jump = IsJumpOpcode(op);
 
         /* Check basic jump opcodes, which may or may not have a fallthrough. */
         if (jump) {
             /* Some opcodes behave differently on their branching path. */