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

Unified Diff: pkg/compiler/lib/src/js_backend/backend.dart

Issue 2302363003: No longer store the compilation-context in WorkItem. (Closed)
Patch Set: ... lint Created 4 years, 3 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « pkg/compiler/lib/src/enqueue.dart ('k') | pkg/compiler/lib/src/js_backend/enqueuer.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pkg/compiler/lib/src/js_backend/backend.dart
diff --git a/pkg/compiler/lib/src/js_backend/backend.dart b/pkg/compiler/lib/src/js_backend/backend.dart
index be81b8a1634a1f2bbc640cc838ad5ff7b3884c56..d155aaa35c255c36096b0a10bbf36e906f059b28 100644
--- a/pkg/compiler/lib/src/js_backend/backend.dart
+++ b/pkg/compiler/lib/src/js_backend/backend.dart
@@ -17,7 +17,6 @@ import '../common/names.dart' show Identifiers, Selectors, Uris;
import '../common/registry.dart' show EagerRegistry, Registry;
import '../common/resolution.dart' show Frontend, Resolution, ResolutionImpact;
import '../common/tasks.dart' show CompilerTask;
-import '../common/work.dart' show ItemCompilationContext;
import '../compiler.dart' show Compiler;
import '../constants/constant_system.dart';
import '../constants/expressions.dart';
@@ -78,11 +77,6 @@ part 'runtime_types.dart';
const VERBOSE_OPTIMIZER_HINTS = false;
-class JavaScriptItemCompilationContext extends ItemCompilationContext {
- final Set<HInstruction> boundsChecked = new Set<HInstruction>();
- final Set<HInstruction> allocatedFixedLists = new Set<HInstruction>();
-}
-
abstract class FunctionCompiler {
/// Generates JavaScript code for `work.element`.
jsAst.Fun compile(CodegenWorkItem work);
@@ -1303,10 +1297,6 @@ class JavaScriptBackend extends Backend {
needToInitializeDispatchProperty = true;
}
- JavaScriptItemCompilationContext createItemCompilationContext() {
- return new JavaScriptItemCompilationContext();
- }
-
void enqueueHelpers(ResolutionEnqueuer world, Registry registry) {
assert(helpers.interceptorsLibrary != null);
// TODO(ngeoffray): Not enqueuing those two classes currently make
@@ -1567,8 +1557,7 @@ class JavaScriptBackend extends Backend {
CodegenEnqueuer get codegenEnqueuer => compiler.enqueuer.codegen;
CodegenEnqueuer createCodegenEnqueuer(Compiler compiler) {
- return new CodegenEnqueuer(compiler, createItemCompilationContext,
- const TreeShakingEnqueuerStrategy());
+ return new CodegenEnqueuer(compiler, const TreeShakingEnqueuerStrategy());
}
WorldImpact codegen(CodegenWorkItem work) {
« no previous file with comments | « pkg/compiler/lib/src/enqueue.dart ('k') | pkg/compiler/lib/src/js_backend/enqueuer.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698