Commit bd62841e authored by marja's avatar marja Committed by Commit bot

Preparsing inner funcs: Update tests.

1) Fix confusion between for of and for in.

2) If a for loop doesn't declare its variables, no new variables
are introduced (the outer scope variables are used).

3) Add more cases for destructuring for and destructuring catch.

BUG=v8:5501, v8:5516

Review-Url: https://codereview.chromium.org/2614023004
Cr-Commit-Position: refs/heads/master@{#42125}
parent 65537684
......@@ -8648,6 +8648,7 @@ TEST(NoPessimisticContextAllocation) {
} inners[] = {
// Context allocating because we need to:
{"function inner() { my_var; }", true},
{"function inner() { if (true) { let my_var; } my_var; }", true},
{"function inner() { eval(\"foo\"); }", true},
{"function inner() { function inner2() { my_var; } }", true},
{"function inner() { function inner2() { eval(\"foo\"); } }", true},
......@@ -8663,9 +8664,37 @@ TEST(NoPessimisticContextAllocation) {
{"function inner([a] = [my_var]) { }", true},
{"function inner() { function inner2([a] = [my_var]) { } }", true},
{"function inner() { ([a] = [my_var]) => { } }", true},
{"function inner() { try { } catch (my_var) { } my_var; }", true},
{"function inner() { for (my_var in {}) { my_var; } }", true},
{"function inner() { for (my_var in {}) { } }", true},
{"function inner() { for (my_var of []) { my_var; } }", true},
{"function inner() { for (my_var of []) { } }", true},
{"function inner() { for ([a, my_var, b] in {}) { my_var; } }", true},
{"function inner() { for ([a, my_var, b] of []) { my_var; } }", true},
{"function inner() { for ({x: my_var} in {}) { my_var; } }", true},
{"function inner() { for ({x: my_var} of []) { my_var; } }", true},
{"function inner() { for ({my_var} in {}) { my_var; } }", true},
{"function inner() { for ({my_var} of []) { my_var; } }", true},
{"function inner() { for ({y, x: my_var} in {}) { my_var; } }", true},
{"function inner() { for ({y, x: my_var} of []) { my_var; } }", true},
{"function inner() { for ({a, my_var} in {}) { my_var; } }", true},
{"function inner() { for ({a, my_var} of []) { my_var; } }", true},
{"function inner() { for (let my_var in {}) { } my_var; }", true},
{"function inner() { for (let my_var of []) { } my_var; }", true},
{"function inner() { for (let [a, my_var, b] in {}) { } my_var; }", true},
{"function inner() { for (let [a, my_var, b] of []) { } my_var; }", true},
{"function inner() { for (let {x: my_var} in {}) { } my_var; }", true},
{"function inner() { for (let {x: my_var} of []) { } my_var; }", true},
{"function inner() { for (let {my_var} in {}) { } my_var; }", true},
{"function inner() { for (let {my_var} of []) { } my_var; }", true},
{"function inner() { for (let {y, x: my_var} in {}) { } my_var; }", true},
{"function inner() { for (let {y, x: my_var} of []) { } my_var; }", true},
{"function inner() { for (let {a, my_var} in {}) { } my_var; }", true},
{"function inner() { for (let {a, my_var} of []) { } my_var; }", true},
// No pessimistic context allocation:
{"function inner() { var my_var; my_var; }", false},
{"function inner() { var my_var; }", false},
{"function inner() { if (true) { var my_var; } my_var; }", false},
{"function inner() { let my_var; my_var; }", false},
{"function inner() { let my_var; }", false},
{"function inner() { const my_var = 0; my_var; }", false},
......@@ -8741,8 +8770,14 @@ TEST(NoPessimisticContextAllocation) {
{"({my_var}) => { }", false},
{"({my_var} = {my_var: 5}) => my_var;", false},
{"({my_var} = {my_var: 5}) => { }", false},
{"function inner() { try { } catch (my_var) { } }", false},
{"function inner() { class my_var {}; }", false},
{"({a, my_var}) => my_var;", false},
{"({a, my_var}) => { }", false},
{"({a, my_var} = {a: 0, my_var: 5}) => my_var;", false},
{"({a, my_var} = {a: 0, my_var: 5}) => { }", false},
{"({y, x: my_var}) => my_var;", false},
{"({y, x: my_var}) => { }", false},
{"({y, x: my_var} = {y: 0, x: 0}) => my_var;", false},
{"({y, x: my_var} = {y: 0, x: 0}) => { }", false},
{"function inner() { my_var => my_var; }", false},
{"function inner() { my_var => { }}", false},
{"function inner() { (my_var = 5) => my_var; }", false},
......@@ -8761,15 +8796,70 @@ TEST(NoPessimisticContextAllocation) {
{"function inner() { ({my_var}) => { }}", false},
{"function inner() { ({my_var} = {my_var: 5}) => my_var;}", false},
{"function inner() { ({my_var} = {my_var: 5}) => { }}", false},
{"function inner() { ({a, my_var}) => my_var;}", false},
{"function inner() { ({a, my_var}) => { }}", false},
{"function inner() { ({a, my_var} = {a: 0, my_var: 5}) => my_var;}",
false},
{"function inner() { ({a, my_var} = {a: 0, my_var: 5}) => { }}", false},
{"function inner() { ({y, x: my_var}) => my_var;}", false},
{"function inner() { ({y, x: my_var}) => { }}", false},
{"function inner() { ({y, x: my_var} = {y: 0, x: 0}) => my_var;}", false},
{"function inner() { ({y, x: my_var} = {y: 0, x: 0}) => { }}", false},
// In the following cases we still context allocate pessimistically:
{"function inner() { function my_var() {} my_var; }", true},
{"function inner() { if (true) { function my_var() {} } my_var; }",
true},
{"function inner() { try { } catch (my_var) { my_var; } }", true},
{"function inner() { for (my_var of {}) { my_var; } }", true},
{"function inner() { for (my_var of {}) { } }", true},
{"function inner() { for (my_var in []) { my_var; } }", true},
{"function inner() { for (my_var in []) { } }", true},
{"function inner() { try { } catch ([a, my_var, b]) { my_var; } }", true},
{"function inner() { try { } catch ({x: my_var}) { my_var; } }", true},
{"function inner() { try { } catch ({y, x: my_var}) { my_var; } }", true},
{"function inner() { try { } catch ({my_var}) { my_var; } }", true},
{"function inner() { for (let my_var = 0; my_var < 1; ++my_var) { my_var "
"} }",
true},
{"function inner() { for (let a = 0, my_var = 0; my_var < 1; ++my_var) { "
"my_var } }",
true},
{"function inner() { for (let my_var in {}) { my_var; } }", true},
{"function inner() { for (let my_var in {}) { } }", true},
{"function inner() { for (let my_var of []) { my_var; } }", true},
{"function inner() { for (let my_var of []) { } }", true},
{"function inner() { for (let [a, my_var, b] in {}) { my_var; } }", true},
{"function inner() { for (let [a, my_var, b] of []) { my_var; } }", true},
{"function inner() { for (let {x: my_var} in {}) { my_var; } }", true},
{"function inner() { for (let {x: my_var} of []) { my_var; } }", true},
{"function inner() { for (let {my_var} in {}) { my_var; } }", true},
{"function inner() { for (let {my_var} of []) { my_var; } }", true},
{"function inner() { for (let {y, x: my_var} in {}) { my_var; } }", true},
{"function inner() { for (let {y, x: my_var} of []) { my_var; } }", true},
{"function inner() { for (let {a, my_var} in {}) { my_var; } }", true},
{"function inner() { for (let {a, my_var} of []) { my_var; } }", true},
{"function inner() { for (var my_var in {}) { my_var; } }", true},
{"function inner() { for (var my_var in {}) { } }", true},
{"function inner() { for (var my_var of []) { my_var; } }", true},
{"function inner() { for (var my_var of []) { } }", true},
{"function inner() { for (var [a, my_var, b] in {}) { my_var; } }", true},
{"function inner() { for (var [a, my_var, b] of []) { my_var; } }", true},
{"function inner() { for (var {x: my_var} in {}) { my_var; } }", true},
{"function inner() { for (var {x: my_var} of []) { my_var; } }", true},
{"function inner() { for (var {my_var} in {}) { my_var; } }", true},
{"function inner() { for (var {my_var} of []) { my_var; } }", true},
{"function inner() { for (var {y, x: my_var} in {}) { my_var; } }", true},
{"function inner() { for (var {y, x: my_var} of []) { my_var; } }", true},
{"function inner() { for (var {a, my_var} in {}) { my_var; } }", true},
{"function inner() { for (var {a, my_var} of []) { my_var; } }", true},
{"function inner() { for (var my_var in {}) { } my_var; }", true},
{"function inner() { for (var my_var of []) { } my_var; }", true},
{"function inner() { for (var [a, my_var, b] in {}) { } my_var; }", true},
{"function inner() { for (var [a, my_var, b] of []) { } my_var; }", true},
{"function inner() { for (var {x: my_var} in {}) { } my_var; }", true},
{"function inner() { for (var {x: my_var} of []) { } my_var; }", true},
{"function inner() { for (var {my_var} in {}) { } my_var; }", true},
{"function inner() { for (var {my_var} of []) { } my_var; }", true},
{"function inner() { for (var {y, x: my_var} in {}) { } my_var; }", true},
{"function inner() { for (var {y, x: my_var} of []) { } my_var; }", true},
{"function inner() { for (var {a, my_var} in {}) { } my_var; }", true},
{"function inner() { for (var {a, my_var} of []) { } my_var; }", true},
{"function inner() { class my_var {}; my_var }", true},
};
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment