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

Side by Side Diff: pkg/compiler/lib/src/use_unused_api.dart

Issue 2857373002: Handle more constants in Constantifier (Closed)
Patch Set: Updated cf. comments. Created 3 years, 7 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
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 /// This file use methods that aren't used by dart2js.dart, but that we wish to 5 /// This file use methods that aren't used by dart2js.dart, but that we wish to
6 /// keep anyway. This might be general API that isn't currently in use, 6 /// keep anyway. This might be general API that isn't currently in use,
7 /// debugging aids, or API only used for testing (see TODO below). 7 /// debugging aids, or API only used for testing (see TODO below).
8 8
9 library dart2js.use_unused_api; 9 library dart2js.use_unused_api;
10 10
(...skipping 78 matching lines...) Expand 10 before | Expand all | Expand 10 after
89 @override 89 @override
90 visitRedirectingGenerative( 90 visitRedirectingGenerative(
91 constants.RedirectingGenerativeConstantConstructor constructor, arg) {} 91 constants.RedirectingGenerativeConstantConstructor constructor, arg) {}
92 } 92 }
93 93
94 void useConstant( 94 void useConstant(
95 [constants.ConstantValue constant, 95 [constants.ConstantValue constant,
96 constants.ConstantExpression expression, 96 constants.ConstantExpression expression,
97 constants.ConstructedConstantExpression constructedConstant, 97 constants.ConstructedConstantExpression constructedConstant,
98 constants.ConstantSystem cs, 98 constants.ConstantSystem cs,
99 constants.Environment env]) { 99 constants.EvaluationEnvironment env]) {
100 constant.isObject; 100 constant.isObject;
101 cs.isBool(constant); 101 cs.isBool(constant);
102 constructedConstant.computeInstanceType(null); 102 constructedConstant.computeInstanceType(null);
103 constructedConstant.computeInstanceFields(null); 103 constructedConstant.computeInstanceFields(null);
104 expression.evaluate(null, null); 104 expression.evaluate(null, null);
105 new NullConstantConstructorVisitor() 105 new NullConstantConstructorVisitor()
106 ..visit(null, null) 106 ..visit(null, null)
107 ..visitGenerative(null, null) 107 ..visitGenerative(null, null)
108 ..visitRedirectingFactory(null, null) 108 ..visitRedirectingFactory(null, null)
109 ..visitRedirectingGenerative(null, null); 109 ..visitRedirectingGenerative(null, null);
(...skipping 180 matching lines...) Expand 10 before | Expand all | Expand 10 after
290 operators.UnaryOperator.fromKind(null); 290 operators.UnaryOperator.fromKind(null);
291 operators.BinaryOperator.fromKind(null); 291 operators.BinaryOperator.fromKind(null);
292 new semantic_visitor.BulkSendVisitor()..apply(null, null); 292 new semantic_visitor.BulkSendVisitor()..apply(null, null);
293 new semantic_visitor.TraversalVisitor(null).apply(null, null); 293 new semantic_visitor.TraversalVisitor(null).apply(null, null);
294 new semantic_visitor.BulkDeclarationVisitor().apply(null, null); 294 new semantic_visitor.BulkDeclarationVisitor().apply(null, null);
295 } 295 }
296 296
297 useDeferred([deferred_load.DeferredLoadTask task]) { 297 useDeferred([deferred_load.DeferredLoadTask task]) {
298 task.dump(); 298 task.dump();
299 } 299 }
OLDNEW
« no previous file with comments | « pkg/compiler/lib/src/kernel/kernel_strategy.dart ('k') | tests/compiler/dart2js/constant_expression_evaluate_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698