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

Unified Diff: frog/tests/leg/src/TypeInference5Test.dart

Issue 10139012: Refactor types in ssa nodes. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: More fixes. Created 8 years, 8 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
Index: frog/tests/leg/src/TypeInference5Test.dart
diff --git a/frog/tests/leg/src/TypeInference5Test.dart b/frog/tests/leg/src/TypeInference5Test.dart
new file mode 100644
index 0000000000000000000000000000000000000000..6a04574f147083cae14667fe503e951e2c1d3bcf
--- /dev/null
+++ b/frog/tests/leg/src/TypeInference5Test.dart
@@ -0,0 +1,26 @@
+// Copyright (c) 2011, the Dart project authors. Please see the AUTHORS file
+// for details. All rights reserved. Use of this source code is governed by a
+// BSD-style license that can be found in the LICENSE file.
+
+#import("compiler_helper.dart");
+
+final String TEST_ONE = @"""
+foo(j) {
+ var a = [1, 2, 3];
+ for (var i = j; i < 3; i++) {
+ a[i];
+ }
+}
+""";
+
+main() {
+ String generated = compile(TEST_ONE, 'foo');
+
+ // Don't test for illegal argument exception. This means that the arguments
ngeoffray 2012/04/25 11:41:13 ditto
floitsch 2012/04/25 19:52:15 Done.
+ // are known to be integers.
+ Expect.isFalse(generated.contains('iae'));
+ // Also make sure that we are not just in bailout mode without speculative
+ // types by grepping for the integer-bailout check on argument j.
+ RegExp regexp = new RegExp(getIntTypeCheck('j'));
+ Expect.isTrue(regexp.hasMatch(generated));
+}

Powered by Google App Engine
This is Rietveld 408576698