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

Side by Side Diff: test/mjsunit/compiler/regress-628403.js

Issue 2159603002: [turbofan] In effect control linearizer, only delay effect phi update for loops. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Fix comment Created 4 years, 5 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 the V8 project authors. All rights reserved. 1 // Copyright 2016 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // Flags: --allow-natives-syntax 5 // Flags: --allow-natives-syntax
6 6
7 var a = [0, ""]; 7 var dothrow = false;
8 a[0] = 0;
9 8
10 function g(array) { 9 function g() {
11 array[1] = undefined; 10 if (dothrow) throw 1;
12 } 11 }
13 12
14 function f() { 13 function f(a) {
15 g(function() {}); 14 try {
16 g(a); 15 g();
16 } catch(e) {
17 if (typeof e !== 'number' && e !== 1) throw e;
18 return a[0];
19 }
17 } 20 }
18 21
22 %NeverOptimizeFunction(g);
23 f();
19 f(); 24 f();
20 %OptimizeFunctionOnNextCall(f); 25 %OptimizeFunctionOnNextCall(f);
21 f(); 26 dothrow = true;
27 assertEquals(42, f([42]));
OLDNEW
« src/compiler/effect-control-linearizer.cc ('K') | « src/compiler/effect-control-linearizer.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698