Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6)

Unified Diff: test/mjsunit/es7/syntactic-tail-call.js

Issue 1928203002: [es8] More spec compliant syntactic tail calls implementation. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: test/mjsunit/es7/syntactic-tail-call.js
diff --git a/test/mjsunit/es7/syntactic-tail-call.js b/test/mjsunit/es7/syntactic-tail-call.js
index b65fc450590afff8f051e1323dad6a4ec38ceede..6373717191d1486ca27cb0768f0ba297cce8d809 100644
--- a/test/mjsunit/es7/syntactic-tail-call.js
+++ b/test/mjsunit/es7/syntactic-tail-call.js
@@ -223,15 +223,15 @@ function f_153(expected_call_stack, a) {
// Tail calling via various expressions.
(function() {
function g1(a) {
- return continue f([f, g1, test], false) || f([f, test], true);
+ return f([f, g1, test], false) || continue f([f, test], true);
}
function g2(a) {
- return continue f([f, g2, test], true) && f([f, test], true);
+ return f([f, g2, test], true) && continue f([f, test], true);
}
function g3(a) {
- return continue f([f, g3, test], 13), f([f, test], 153);
+ return f([f, g3, test], 13), continue f([f, test], 153);
}
function test() {
@@ -404,10 +404,9 @@ function f_153(expected_call_stack, a) {
}
function g3(a) {
- var closure = () => continue f([f, closure, test], true)
- ? f_153([f_153, test])
- : f_153([f_153, test]);
-
+ var closure = () => f([f, closure, test], true)
+ ? continue f_153([f_153, test])
+ : f_153([f_153, test]);
rossberg 2016/05/02 10:59:40 Shouldn't you need a `continue` here, too, to be e
Igor Sheludko 2016/05/04 10:28:55 The call f([f, closure, test], true) returns true.
return continue closure();
}
« src/parsing/parser-base.h ('K') | « test/message/syntactic-tail-call-without-return.out ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698