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

Unified Diff: test/mjsunit/es6/regress/regress-650172.js

Issue 2368323002: [builtins] adapt arguments for Builtins::kIteratorPrototypeIterator (Closed)
Patch Set: Created 4 years, 3 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
« no previous file with comments | « src/bootstrapper.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/mjsunit/es6/regress/regress-650172.js
diff --git a/test/message/generators-throw1.js b/test/mjsunit/es6/regress/regress-650172.js
similarity index 59%
copy from test/message/generators-throw1.js
copy to test/mjsunit/es6/regress/regress-650172.js
index b4d404a1bf4d1e8f46c669c79f9ececd6f1ee64f..1b7d30a7ee34a058bc2ade566c2821f445abfc6b 100644
--- a/test/message/generators-throw1.js
+++ b/test/mjsunit/es6/regress/regress-650172.js
@@ -2,6 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-function* f() { yield }
+// Flags: --ignition
Benedikt Meurer 2016/09/26 18:30:31 Does it also repro with --ignition-staging? If so,
caitp 2016/09/26 21:51:47 It does if the division is there to cause the x87
Benedikt Meurer 2016/09/27 03:33:24 Please nuke the Flags: if it also crashes with --i
caitp 2016/09/27 09:09:03 So, you're saying you'd prefer `Flags: --ignition-
Benedikt Meurer 2016/09/27 09:14:29 Sorry for the confusion: We run --ignition-staging
caitp 2016/09/27 09:18:23 I see, done.
-f().throw(42);
+var iterator = [].entries().__proto__.__proto__[Symbol.iterator];
Benedikt Meurer 2016/09/26 18:30:31 How about using Array.prototype[Symbol.iterator]._
caitp 2016/09/26 21:51:47 SGTM
caitp 2016/09/26 22:12:52 Actually, doing that doesn't hit the crash. [].en
Benedikt Meurer 2016/09/27 03:33:24 Acknowledged.
+print(1/iterator(-1E-300));
Benedikt Meurer 2016/09/26 18:30:31 I guess you don't need the print or the division,
caitp 2016/09/26 21:51:47 well, the division causes an x87 PF (because not e
Benedikt Meurer 2016/09/27 03:33:24 Acknowledged.
« no previous file with comments | « src/bootstrapper.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698