708777 - IonMonkey: Assertion failure: unexpected type, at Lowering.cpp:538

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Jan de Mooij [:jandem]]

jit-tests/tests/jaeger/recompile/undef.js asserts. Here's a reduced testcase without callgname:
--
function local() {
    var x;
    x++;
    return x;
}
local();
--
Asserts when lowering MToInt32(undefined).

Comment 1

[David Anderson [:dvander] - inactive, e-mail if emergency]

Attachment: fix

Comment 2

[Brian Hackett [Laid off!]]

Comment on attachment 594364 [details] [diff] [review]
fix

Review of attachment 594364 [details] [diff] [review]:
-----------------------------------------------------------------

::: js/src/shell/js.cpp
@@ +5461,5 @@
>          || !op.addStringOption('\0', "ion-gvn", "[mode]",
>                                 "Specify Ion global value numbering:\n"
>                                 "  off: disable GVN\n"
> +                               "  pessimistic: (default) use pessimistic GVN\n"
> +                               "  optimistic: use optimistic GVN")

Intended for this patch?

Comment 3

[David Anderson [:dvander] - inactive, e-mail if emergency]

http://hg.mozilla.org/projects/ionmonkey/rev/55fe4b0baa59