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

Unified Diff: tests/compiler/dart2js/call_site_simple_type_inferer_test.dart

Issue 2577423002: Further reduce use of Compiler.closedWorld. (Closed)
Patch Set: Created 4 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « tests/compiler/dart2js/assert_message_throw_test.dart ('k') | tests/compiler/dart2js/class_set_test.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/compiler/dart2js/call_site_simple_type_inferer_test.dart
diff --git a/tests/compiler/dart2js/call_site_simple_type_inferer_test.dart b/tests/compiler/dart2js/call_site_simple_type_inferer_test.dart
index f12fe382cd309b4e4da7cd62d9cb493115b64bd4..44ea2ba36dbf1d3142e5f7b7f8bcadc2f4856579 100644
--- a/tests/compiler/dart2js/call_site_simple_type_inferer_test.dart
+++ b/tests/compiler/dart2js/call_site_simple_type_inferer_test.dart
@@ -2,8 +2,9 @@
// 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 'package:async_helper/async_helper.dart';
+import 'package:compiler/src/types/masks.dart';
import 'package:expect/expect.dart';
-import "package:async_helper/async_helper.dart";
import 'compiler_helper.dart';
import 'type_mask_test_helper.dart';
@@ -206,109 +207,85 @@ const String TEST_18 = r"""
}
""";
-void doTest(String test, bool enableInlining, Function f) {
+typedef List<TypeMask> TestCallback(CommonMasks masks);
+
+void doTest(String test, bool enableInlining, TestCallback f) {
compileAndFind(test, 'A', 'x', enableInlining, (compiler, element) {
- var expectedTypes = f(compiler);
+ var inferrer = compiler.globalInference.typesInferrerInternal;
+ var closedWorld = inferrer.closedWorld;
+ var expectedTypes = f(closedWorld.commonMasks);
var signature = element.functionSignature;
int index = 0;
- var inferrer = compiler.globalInference.typesInferrerInternal;
signature.forEachParameter((Element element) {
Expect.equals(expectedTypes[index++],
- simplify(inferrer.getTypeOfElement(element), compiler), test);
+ simplify(inferrer.getTypeOfElement(element), closedWorld), test);
});
Expect.equals(index, expectedTypes.length);
});
}
-void runTest(String test, Function f) {
+void runTest(String test, TestCallback f) {
doTest(test, false, f);
doTest(test, true, f);
}
-subclassOfInterceptor(compiler) {
- return findTypeMask(compiler, 'Interceptor', 'nonNullSubclass');
-}
-
void test() {
- runTest(TEST_1, (compiler) => [compiler.closedWorld.commonMasks.stringType]);
- runTest(TEST_2, (compiler) => [compiler.closedWorld.commonMasks.uint31Type]);
- runTest(TEST_3, (compiler) => [compiler.closedWorld.commonMasks.intType]);
- runTest(TEST_4, (compiler) => [compiler.closedWorld.commonMasks.numType]);
- runTest(TEST_5, (compiler) => [compiler.closedWorld.commonMasks.numType]);
- runTest(TEST_6, (compiler) => [compiler.closedWorld.commonMasks.numType]);
- runTest(TEST_7a, (compiler) => [subclassOfInterceptor(compiler)]);
- runTest(
- TEST_7b,
- (compiler) =>
- [compiler.closedWorld.commonMasks.dynamicType.nonNullable()]);
+ runTest(TEST_1, (commonMasks) => [commonMasks.stringType]);
+ runTest(TEST_2, (commonMasks) => [commonMasks.uint31Type]);
+ runTest(TEST_3, (commonMasks) => [commonMasks.intType]);
+ runTest(TEST_4, (commonMasks) => [commonMasks.numType]);
+ runTest(TEST_5, (commonMasks) => [commonMasks.numType]);
+ runTest(TEST_6, (commonMasks) => [commonMasks.numType]);
+ runTest(TEST_7a, (commonMasks) => [commonMasks.interceptorType]);
+ runTest(TEST_7b, (commonMasks) => [commonMasks.dynamicType.nonNullable()]);
runTest(
TEST_8,
- (compiler) => [
- compiler.closedWorld.commonMasks.uint31Type,
- subclassOfInterceptor(compiler),
- compiler.closedWorld.commonMasks.dynamicType.nonNullable()
- ]);
- runTest(
- TEST_9,
- (compiler) => [
- compiler.closedWorld.commonMasks.uint31Type,
- compiler.closedWorld.commonMasks.uint31Type
- ]);
- runTest(
- TEST_10,
- (compiler) => [
- compiler.closedWorld.commonMasks.uint31Type,
- compiler.closedWorld.commonMasks.uint31Type
+ (commonMasks) => [
+ commonMasks.uint31Type,
+ commonMasks.interceptorType,
+ commonMasks.dynamicType.nonNullable()
]);
+ runTest(TEST_9,
+ (commonMasks) => [commonMasks.uint31Type, commonMasks.uint31Type]);
+ runTest(TEST_10,
+ (commonMasks) => [commonMasks.uint31Type, commonMasks.uint31Type]);
runTest(
TEST_11,
- (compiler) =>
- [subclassOfInterceptor(compiler), subclassOfInterceptor(compiler)]);
+ (commonMasks) =>
+ [commonMasks.interceptorType, commonMasks.interceptorType]);
- runTest(
- TEST_12,
- (compiler) => [
- compiler.closedWorld.commonMasks.stringType,
- compiler.closedWorld.commonMasks.uint31Type
- ]);
+ runTest(TEST_12,
+ (commonMasks) => [commonMasks.stringType, commonMasks.uint31Type]);
- runTest(TEST_13, (compiler) => [compiler.closedWorld.commonMasks.numType]);
+ runTest(TEST_13, (commonMasks) => [commonMasks.numType]);
- runTest(
- TEST_14,
- (compiler) => [
- compiler.closedWorld.commonMasks.uint31Type,
- compiler.closedWorld.commonMasks.stringType
- ]);
+ runTest(TEST_14,
+ (commonMasks) => [commonMasks.uint31Type, commonMasks.stringType]);
runTest(
- TEST_15,
- (compiler) => [
- compiler.closedWorld.commonMasks.stringType,
- compiler.closedWorld.commonMasks.boolType
- ]);
+ TEST_15, (commonMasks) => [commonMasks.stringType, commonMasks.boolType]);
runTest(
TEST_16,
- (compiler) => [
- compiler.closedWorld.commonMasks.uint31Type,
- compiler.closedWorld.commonMasks.uint31Type,
- compiler.closedWorld.commonMasks.stringType
+ (commonMasks) => [
+ commonMasks.uint31Type,
+ commonMasks.uint31Type,
+ commonMasks.stringType
]);
runTest(
TEST_17,
- (compiler) => [
- compiler.closedWorld.commonMasks.uint31Type,
- compiler.closedWorld.commonMasks.boolType,
- compiler.closedWorld.commonMasks.doubleType
+ (commonMasks) => [
+ commonMasks.uint31Type,
+ commonMasks.boolType,
+ commonMasks.doubleType
]);
runTest(
TEST_18,
- (compiler) =>
- [subclassOfInterceptor(compiler), subclassOfInterceptor(compiler)]);
+ (commonMasks) =>
+ [commonMasks.interceptorType, commonMasks.interceptorType]);
}
void main() {
« no previous file with comments | « tests/compiler/dart2js/assert_message_throw_test.dart ('k') | tests/compiler/dart2js/class_set_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698