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

Unified 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 side-by-side diff with in-line comments
Download patch
« 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 »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/language/inline_getter_test.dart
===================================================================
--- tests/language/inline_getter_test.dart (revision 24922)
+++ tests/language/inline_getter_test.dart (working copy)
@@ -4,6 +4,7 @@
// Test inlining of instance getters.
// Three classes access always the same field. Optimize method foo and inline
// getter for classes 'A' and 'B'. Call later via 'C' and cause deoptimization.
+// VMOptions=--optimization-counter-threshold=10 --no-use-osr
import "package:expect/expect.dart";
@@ -28,13 +29,13 @@
var a = new A(1);
var b = new B();
int sum = 0;
- for (int i = 0; i < 5000; i++) {
+ for (int i = 0; i < 20; i++) {
sum += a.foo();
sum += b.foo();
}
var c = new C();
sum += c.foo(); // <-- Deoptimizing.
- Expect.equals(15010, sum);
+ Expect.equals(70, sum);
}
}
« 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