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

Side by Side Diff: tests/language/inline_effect_context_test.dart

Issue 18563006: More tests updated to optimize code (Part 2). (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: Created 7 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 // Test inlining of simple function with control flow in an effect context. 4 // Test inlining of simple function with control flow in an effect context.
5 // Optimize function foo with instance of A and inlined function bar. Call later 5 // Optimize function foo with instance of A and inlined function bar. Call later
6 // with instance of B and cause deoptimization. 6 // with instance of B and cause deoptimization.
7 // VMOptions=--optimization-counter-threshold=10 --no-use-osr
7 8
8 import "package:expect/expect.dart"; 9 import "package:expect/expect.dart";
9 10
10 class A { 11 class A {
11 var x = 1; 12 var x = 1;
12 } 13 }
13 14
14 class B { 15 class B {
15 var x = 0; 16 var x = 0;
16 } 17 }
17 18
18 int bar(o) { 19 int bar(o) {
19 if (o.x > 0) { // <-- Deoptimize from inner frame. 20 if (o.x > 0) { // <-- Deoptimize from inner frame.
20 return 1; 21 return 1;
21 } else { 22 } else {
22 return 0; 23 return 0;
23 } 24 }
24 } 25 }
25 26
26 int foo(o) { 27 int foo(o) {
27 bar(o); // <-- Used in an effect context. 28 bar(o); // <-- Used in an effect context.
28 return 1; 29 return 1;
29 } 30 }
30 31
31 main() { 32 main() {
32 var o = new A(); 33 var o = new A();
33 int sum = 0; 34 int sum = 0;
34 for (int i = 0; i < 5000; i++) sum += foo(o); 35 for (int i = 0; i < 20; i++) sum += foo(o);
35 o = new B(); 36 o = new B();
36 sum += foo(o); // <-- Cause deoptimization of bar within foo. 37 sum += foo(o); // <-- Cause deoptimization of bar within foo.
37 Expect.equals(5001, sum); 38 Expect.equals(21, sum);
38 } 39 }
OLDNEW
« no previous file with comments | « tests/language/inline_closure_with_constant_arguments_test.dart ('k') | tests/language/inline_getter_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698