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

Unified Diff: lib/compiler/implementation/ssa/variable_allocator.dart

Issue 10477010: Fix some editor warnings. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 8 years, 6 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: lib/compiler/implementation/ssa/variable_allocator.dart
diff --git a/lib/compiler/implementation/ssa/variable_allocator.dart b/lib/compiler/implementation/ssa/variable_allocator.dart
index 6defcc201a0f4a1ee4434db39e0effc5b4fa7549..33fd7517b21f3498821f57ef59903fddbbe96020 100644
--- a/lib/compiler/implementation/ssa/variable_allocator.dart
+++ b/lib/compiler/implementation/ssa/variable_allocator.dart
@@ -29,13 +29,13 @@ class LiveInterval {
LiveInterval() : ranges = <LiveRange>[];
/**
- * Update all ranges that are contained in [start, end[ to
- * die at [end].
+ * Update all ranges that are contained in [from, to[ to
+ * die at [to].
*/
- void loopUpdate(int start, int end) {
+ void loopUpdate(int from, int to) {
for (LiveRange range in ranges) {
- if (start <= range.start && range.end < end) {
- range.end = end;
+ if (from <= range.start && range.end < to) {
+ range.end = to;
}
}
}
@@ -441,7 +441,7 @@ class VariableNamer {
// If the name is null, then the checked input is being
// generated at use site, and we don't need a name for the check
// instruction.
- if (name == null) return;
+ if (name == null) return null;
} else if (instruction is HParameterValue) {
HParameterValue parameter = instruction;
name = parameterNames[parameter.element];
« lib/compiler/implementation/ssa/nodes.dart ('K') | « lib/compiler/implementation/ssa/nodes.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698