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

Unified Diff: tests/kernel/unsorted/finally_contexts_test.dart

Issue 2434123003: Merge more Kernel infrastructure from kernel_sdk SDK fork. (Closed)
Patch Set: address more comments Created 4 years, 2 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: tests/kernel/unsorted/finally_contexts_test.dart
diff --git a/tests/kernel/unsorted/finally_contexts_test.dart b/tests/kernel/unsorted/finally_contexts_test.dart
new file mode 100644
index 0000000000000000000000000000000000000000..c844d809e12924efa95a69c033fa8e59111c1ad2
--- /dev/null
+++ b/tests/kernel/unsorted/finally_contexts_test.dart
@@ -0,0 +1,47 @@
+// Copyright (c) 2016, 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 'expect.dart';
+
+// Test that the context depth is correct in the presence of control flow,
+// specifically branching and joining in the presence of break. The
+// implementation uses the context depth after the else block of an if/then/else
+// as the context depth at the join point. This test has an extra context
+// allocated in the (untaken) else branch so it tests that compiling the
+// (untaken) break properly tracks the context depth.
+
+test(list) {
+ // The loops force creation of a new context, otherwise context allocated
+ // variables might be hoisted to an outer context.
+ do {
+ if (list.length > 1) {
+ do {
+ var sum = 0;
+ addem() {
+ for (var x in list) sum += x;
+ }
+ addem();
+ Expect.isTrue(sum == 15);
+ L: if (sum != 15) {
+ // Unreachable.
+ do {
+ var product = 1;
+ multiplyem() {
+ for (var x in list) product *= n;
+ }
+ multiplyem();
+ Expect.isTrue(false);
+ break L;
+ } while (false);
+ }
+ } while (false);
+ }
+ } while (false);
+ Expect.isTrue(list.length == 5);
+}
+
+
+main() {
+ test([1, 2, 3, 4, 5]);
+}

Powered by Google App Engine
This is Rietveld 408576698