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

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

Issue 11275122: Merge revision r14430 to trunk. Do not evaluate arguments of an instance send if that instance sen… (Closed) Base URL: http://dart.googlecode.com/svn/trunk/
Patch Set: Created 8 years, 1 month 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
« no previous file with comments | « dart/lib/compiler/implementation/ssa/builder.dart ('k') | dart/tools/VERSION » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 that when inlining A.foo, we're not evaluating the argument 4 // Test that when inlining A.foo, we're not evaluating the argument
5 // twice. 5 // twice.
6 6
7 class A { 7 class A {
8 var field = 0; 8 var field = 0;
9 9
10 foo(b) { 10 foo(b) {
11 Expect.equals(0, b); 11 Expect.equals(0, b);
12 Expect.equals(0, b); 12 Expect.equals(0, b);
13 } 13 }
14 14
15 bar() { 15 bar() {
16 foo(field++); 16 foo(field++);
17 } 17 }
18 } 18 }
19 19
20 main() { 20 main() {
21 var a = new A(); 21 var a = new A();
22 a.bar(); 22 a.bar();
23 Expect.equals(1, a.field); 23 Expect.equals(1, a.field);
24 } 24 }
OLDNEW
« no previous file with comments | « dart/lib/compiler/implementation/ssa/builder.dart ('k') | dart/tools/VERSION » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698