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

Side by Side Diff: tests/language/inline_getter_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) 2011, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2011, 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 instance getters. 4 // Test inlining of instance getters.
5 // Three classes access always the same field. Optimize method foo and inline 5 // Three classes access always the same field. Optimize method foo and inline
6 // getter for classes 'A' and 'B'. Call later via 'C' and cause deoptimization. 6 // getter for classes 'A' and 'B'. Call later via 'C' 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 int f; 12 int f;
12 A(this.f) {} 13 A(this.f) {}
13 int foo() { 14 int foo() {
14 return f; // <-- inline getter for classes 'A' and 'B'. 15 return f; // <-- inline getter for classes 'A' and 'B'.
15 } 16 }
16 } 17 }
17 18
18 class B extends A { 19 class B extends A {
19 B() : super(2) {} 20 B() : super(2) {}
20 } 21 }
21 22
22 class C extends A { 23 class C extends A {
23 C() : super(10) {} 24 C() : super(10) {}
24 } 25 }
25 26
26 class InlineGetterTest { 27 class InlineGetterTest {
27 static testMain() { 28 static testMain() {
28 var a = new A(1); 29 var a = new A(1);
29 var b = new B(); 30 var b = new B();
30 int sum = 0; 31 int sum = 0;
31 for (int i = 0; i < 5000; i++) { 32 for (int i = 0; i < 20; i++) {
32 sum += a.foo(); 33 sum += a.foo();
33 sum += b.foo(); 34 sum += b.foo();
34 } 35 }
35 var c = new C(); 36 var c = new C();
36 sum += c.foo(); // <-- Deoptimizing. 37 sum += c.foo(); // <-- Deoptimizing.
37 Expect.equals(15010, sum); 38 Expect.equals(70, sum);
38 } 39 }
39 } 40 }
40 41
41 main() { 42 main() {
42 InlineGetterTest.testMain(); 43 InlineGetterTest.testMain();
43 } 44 }
OLDNEW
« no previous file with comments | « tests/language/inline_effect_context_test.dart ('k') | tests/language/inline_test_context_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698