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

Side by Side Diff: tools/testing/dart/runtime_configuration.dart

Issue 1077823002: Adds a simarmv5te build and test target. (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: Created 5 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2014, 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 library runtime_configuration; 5 library runtime_configuration;
6 6
7 import 'compiler_configuration.dart' show 7 import 'compiler_configuration.dart' show
8 CommandArtifact; 8 CommandArtifact;
9 9
10 // TODO(ahe): Remove this import, we can precompute all the values required 10 // TODO(ahe): Remove this import, we can precompute all the values required
(...skipping 151 matching lines...) Expand 10 before | Expand all | Expand 10 after
162 : super._subclass(); 162 : super._subclass();
163 163
164 int computeTimeoutMultiplier({ 164 int computeTimeoutMultiplier({
165 bool isDebug: false, 165 bool isDebug: false,
166 bool isChecked: false, 166 bool isChecked: false,
167 String arch}) { 167 String arch}) {
168 int multiplier = 1; 168 int multiplier = 1;
169 switch (arch) { 169 switch (arch) {
170 case 'simarm': 170 case 'simarm':
171 case 'arm': 171 case 'arm':
172 case' simarmv5te':
172 case 'armv5te': 173 case 'armv5te':
ricow1 2015/04/10 08:23:41 I wonder if we should have an even higher multipli
zra 2015/04/10 16:07:20 In my initial testing, this timeout seemed to work
173 case 'simmips': 174 case 'simmips':
174 case 'mips': 175 case 'mips':
175 case 'simarm64': 176 case 'simarm64':
176 multiplier *= 4; 177 multiplier *= 4;
177 break; 178 break;
178 } 179 }
179 if (isDebug) { 180 if (isDebug) {
180 multiplier *= 2; 181 multiplier *= 2;
181 } 182 }
182 return multiplier; 183 return multiplier;
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
223 class DummyRuntimeConfiguration extends DartVmRuntimeConfiguration { 224 class DummyRuntimeConfiguration extends DartVmRuntimeConfiguration {
224 List<Command> computeRuntimeCommands( 225 List<Command> computeRuntimeCommands(
225 TestSuite suite, 226 TestSuite suite,
226 CommandBuilder commandBuilder, 227 CommandBuilder commandBuilder,
227 CommandArtifact artifact, 228 CommandArtifact artifact,
228 List<String> arguments, 229 List<String> arguments,
229 Map<String, String> environmentOverrides) { 230 Map<String, String> environmentOverrides) {
230 throw "Unimplemented runtime '$runtimeType'"; 231 throw "Unimplemented runtime '$runtimeType'";
231 } 232 }
232 } 233 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698