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

Unified Diff: pkg/compiler/lib/src/ssa/builder.dart

Issue 2854013002: Make JavaScriptBackend.processAnnotations element-model agnostic (Closed)
Patch Set: Fix and check declaration invariant. Created 3 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 side-by-side diff with in-line comments
Download patch
Index: pkg/compiler/lib/src/ssa/builder.dart
diff --git a/pkg/compiler/lib/src/ssa/builder.dart b/pkg/compiler/lib/src/ssa/builder.dart
index de9545a35ea1ab2d55714433dc88d544053282cb..188f2ae047209b71e20443de9a4db9cce4896671 100644
--- a/pkg/compiler/lib/src/ssa/builder.dart
+++ b/pkg/compiler/lib/src/ssa/builder.dart
@@ -412,7 +412,7 @@ class SsaBuilder extends ast.Visitor
// Bail out early if the inlining decision is in the cache and we can't
// inline (no need to check the hard constraints).
bool cachedCanBeInlined =
- inlineCache.canInline(function, insideLoop: insideLoop);
+ inlineCache.canInline(function.declaration, insideLoop: insideLoop);
if (cachedCanBeInlined == false) return false;
bool meetsHardConstraints() {
@@ -534,9 +534,11 @@ class SsaBuilder extends ast.Visitor
functionResolvedAst, maxInliningNodes,
enableUserAssertions: options.enableUserAssertions);
if (canInline) {
- inlineCache.markAsInlinable(function, insideLoop: insideLoop);
+ inlineCache.markAsInlinable(function.declaration,
+ insideLoop: insideLoop);
} else {
- inlineCache.markAsNonInlinable(function, insideLoop: insideLoop);
+ inlineCache.markAsNonInlinable(function.declaration,
+ insideLoop: insideLoop);
}
return canInline;
}
« no previous file with comments | « pkg/compiler/lib/src/resolution/resolution_strategy.dart ('k') | tests/compiler/dart2js/expect_annotations_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698