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

Unified Diff: test/mjsunit/compiler/osr-typing-debug-change.js

Issue 2388303006: Reland of "[turbofan] Osr value typing + dynamic type checks on entry. (patchset #5 id:80001 of htt… (Closed)
Patch Set: Tweaks Created 4 years, 2 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 | « test/cctest/compiler/test-osr.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/mjsunit/compiler/osr-typing-debug-change.js
diff --git a/test/mjsunit/compiler/osr-typing-debug-change.js b/test/mjsunit/compiler/osr-typing-debug-change.js
new file mode 100644
index 0000000000000000000000000000000000000000..03579c0ff0838568fc66460e843327a1d5267e9c
--- /dev/null
+++ b/test/mjsunit/compiler/osr-typing-debug-change.js
@@ -0,0 +1,119 @@
+// Copyright 2016 the V8 project authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+// Flags: --allow-natives-syntax --expose-debug-as debug
+
+var Debug = debug.Debug;
+
+var changed = false;
+
+function listenerSetJToResult(
+ event, exec_state, event_data, data) {
+
+ if (event == Debug.DebugEvent.Break) {
+ var scope = exec_state.frame(1).scope(0);
+ var newval = "result";
+ try {
+ scope.setVariableValue("j", newval);
+ changed = true;
+ } catch(e) {
+ changed = false;
+ }
+ }
+}
+
+Debug.setListener(listenerSetJToResult);
+
+function g() { debugger; }
+%NeverOptimizeFunction(g);
+
+function ChangeSmiConstantAndOsr() {
+ var j = 1;
+ for (var i = 0; i < 4; i++) {
+ if (i == 2) {
+ %OptimizeOsr();
+ g();
+ }
+ }
+ return j;
+}
+var r1 = ChangeSmiConstantAndOsr();
+if (changed) {
+ assertEquals("result", r1);
+} else {
+ assertEquals(1, r1);
+}
+
+function ChangeFloatConstantAndOsr() {
+ var j = 0.1;
+ for (var i = 0; i < 4; i++) {
+ if (i == 2) {
+ %OptimizeOsr();
+ g();
+ }
+ }
+ return j;
+}
+var r2 = ChangeFloatConstantAndOsr();
+if (changed) {
+ assertEquals("result", r2);
+} else {
+ assertEquals(0.1, r2);
+}
+
+function ChangeFloatVarAndOsr() {
+ var j = 0.1;
+ for (var i = 0; i < 4; i++) {
+ j = j + 0.1;
+ if (i == 2) {
+ %OptimizeOsr();
+ g();
+ }
+ }
+ return j;
+}
+var r3 = ChangeFloatVarAndOsr();
+if (changed) {
+ assertEquals("result0.1", r3);
+} else {
+ assertEquals(0.5, r3);
+}
+
+var counter = 0;
+var o = { toString : function() { counter++; return 100; } };
+
+function listenerSetJToObject(
+ event, exec_state, event_data, data) {
+ if (event == Debug.DebugEvent.Break) {
+ var scope = exec_state.frame(1).scope(0);
+ try {
+ scope.setVariableValue("j", o);
+ changed = true;
+ } catch(e) {
+ changed = false;
+ }
+ }
+}
+
+Debug.setListener(listenerSetJToObject);
+
+function ChangeIntVarAndOsr() {
+ var j = 1;
+ for (var i = 0; i < 4; i++) {
+ j = j + 1|0;
+ if (i == 2) {
+ %OptimizeOsr();
+ g();
+ }
+ }
+ return j;
+}
+
+var r4 = ChangeIntVarAndOsr();
+if (changed) {
+ assertEquals(101, r4);
+ assertEquals(1, counter);
+} else {
+ assertEquals(5, r4);
+}
« no previous file with comments | « test/cctest/compiler/test-osr.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698