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

Side by Side Diff: tests/standalone/array_bounds_check_generalization_test.dart

Issue 1514653002: - Assert that we do not hold the timeline lock while entering (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: Created 5 years 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
« no previous file with comments | « runtime/vm/timeline.cc ('k') | no next file » | 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 // 4 //
5 // VMOptions=--optimization_counter_threshold=10 --no-use-osr 5 // We are using --complete-timeline below to ensure that we get timeline events
6 // generated during all phases of compilation and deoptimization.
7 // VMOptions=--optimization_counter_threshold=10 --no-use-osr --complete-timelin e
Florian Schneider 2015/12/10 17:01:46 Could you reproduce the deadlock with this test? I
6 8
7 import "package:expect/expect.dart"; 9 import "package:expect/expect.dart";
8 10
9 test1(a, start, step, N) { 11 test1(a, start, step, N) {
10 var e; 12 var e;
11 for (var i = 0; i < N; i++) { 13 for (var i = 0; i < N; i++) {
12 e = a[start + i * step]; 14 e = a[start + i * step];
13 } 15 }
14 return e; 16 return e;
15 } 17 }
(...skipping 62 matching lines...) Expand 10 before | Expand all | Expand 10 after
78 Expect.equals(a[a.length - 2], test4(a, b)); 80 Expect.equals(a[a.length - 2], test4(a, b));
79 Expect.equals(a[a.length - 2], test5(a, b, 0)); 81 Expect.equals(a[a.length - 2], test5(a, b, 0));
80 Expect.equals(6 , test6(a, 2, 2)); 82 Expect.equals(6 , test6(a, 2, 2));
81 } 83 }
82 84
83 test1(a, 0, 2, a.length ~/ 2); 85 test1(a, 0, 2, a.length ~/ 2);
84 Expect.throws(() => test1(a, 1, 1, a.length)); 86 Expect.throws(() => test1(a, 1, 1, a.length));
85 Expect.throws(() => test2(a, new List(a.length - 1))); 87 Expect.throws(() => test2(a, new List(a.length - 1)));
86 Expect.throws(() => test6(a, 4, 3)); 88 Expect.throws(() => test6(a, 4, 3));
87 } 89 }
OLDNEW
« no previous file with comments | « runtime/vm/timeline.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698