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

Unified Diff: test/mjsunit/regress/regress-v8-5697.js

Issue 2654733004: [tests] Make assertOptimized()/assertUnoptimized() great again. (Closed)
Patch Set: Update debugger.status Created 3 years, 11 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/regress/regress-v8-5697.js
diff --git a/test/mjsunit/regress/regress-v8-5697.js b/test/mjsunit/regress/regress-v8-5697.js
index 5a88614b1355466e36839a3b7a72ee94b18d3897..66f15f2e09883286ab03d49149ce6e377f9be533 100644
--- a/test/mjsunit/regress/regress-v8-5697.js
+++ b/test/mjsunit/regress/regress-v8-5697.js
@@ -9,6 +9,7 @@ function load(o) { return o.x; }
for (var x = 0; x < 1000; ++x) {
load({x});
load({x});
+ % OptimizeFunctionOnNextCall(load);
Michael Starzinger 2017/01/26 12:24:52 nit: Can we drop the whitespace after the "%" sign
Igor Sheludko 2017/01/26 14:12:32 Done.
try { load(); } catch (e) { }
}
@@ -20,6 +21,7 @@ function store(o) { o.x = -1; }
for (var x = 0; x < 1000; ++x) {
store({x});
store({x});
+ % OptimizeFunctionOnNextCall(store);
Michael Starzinger 2017/01/26 12:24:52 nit: Can we drop the whitespace after the "%" sign
Igor Sheludko 2017/01/26 14:12:32 Done.
try { store(); } catch (e) { }
}

Powered by Google App Engine
This is Rietveld 408576698