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

Side by Side Diff: pkg/kernel/lib/target/flutter.dart

Issue 2671653003: Split the Kernel transformations into modular and global ones (Closed)
Patch Set: Revert inadvertent change Created 3 years, 10 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
« no previous file with comments | « pkg/kernel/bin/dartk.dart ('k') | pkg/kernel/lib/target/targets.dart » ('j') | 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) 2016, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2016, 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 library kernel.target.flutter; 4 library kernel.target.flutter;
5 5
6 import '../ast.dart'; 6 import '../ast.dart';
7 import '../transformations/continuation.dart' as cont; 7 import '../transformations/continuation.dart' as cont;
8 import '../transformations/erasure.dart'; 8 import '../transformations/erasure.dart';
9 import '../transformations/sanitize_for_vm.dart'; 9 import '../transformations/sanitize_for_vm.dart';
10 import '../transformations/mixin_full_resolution.dart' as mix; 10 import '../transformations/mixin_full_resolution.dart' as mix;
(...skipping 32 matching lines...) Expand 10 before | Expand all | Expand 10 after
43 'dart:_builtin', 43 'dart:_builtin',
44 'dart:nativewrappers', 44 'dart:nativewrappers',
45 'dart:io', 45 'dart:io',
46 46
47 // Required for flutter. 47 // Required for flutter.
48 'dart:ui', 48 'dart:ui',
49 'dart:jni', 49 'dart:jni',
50 'dart:vmservice_sky', 50 'dart:vmservice_sky',
51 ]; 51 ];
52 52
53 void transformProgram(Program program) { 53 void performModularTransformations(Program program) {
54 new mix.MixinFullResolution().transform(program); 54 new mix.MixinFullResolution().transform(program);
55 }
56
57 void performGlobalTransformations(Program program) {
asgerf 2017/02/02 11:46:00 The transformations here are not global, I think.
Kevin Millikin (Google) 2017/02/02 12:03:48 They're just not rewritten in this change to skip
55 cont.transformProgram(program); 58 cont.transformProgram(program);
56 59
57 // Repair `_getMainClosure()` function in dart:{_builtin,ui} libraries. 60 // Repair `_getMainClosure()` function in dart:{_builtin,ui} libraries.
58 setup_builtin_library.transformProgram(program); 61 setup_builtin_library.transformProgram(program);
59 setup_builtin_library.transformProgram(program, libraryUri: 'dart:ui'); 62 setup_builtin_library.transformProgram(program, libraryUri: 'dart:ui');
60 63
61 if (strongMode) { 64 if (strongMode) {
62 new Erasure().transform(program); 65 new Erasure().transform(program);
63 } 66 }
64 67
65 new SanitizeForVM().transform(program); 68 new SanitizeForVM().transform(program);
66 } 69 }
67 } 70 }
OLDNEW
« no previous file with comments | « pkg/kernel/bin/dartk.dart ('k') | pkg/kernel/lib/target/targets.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698