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

Unified Diff: sdk/lib/_internal/compiler/implementation/ssa/tracer.dart

Issue 17028011: Implement a list tracer phase, that tries to find element types in individual lists. (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: Created 7 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: sdk/lib/_internal/compiler/implementation/ssa/tracer.dart
===================================================================
--- sdk/lib/_internal/compiler/implementation/ssa/tracer.dart (revision 24023)
+++ sdk/lib/_internal/compiler/implementation/ssa/tracer.dart (working copy)
@@ -179,12 +179,18 @@
String temporaryId(HInstruction instruction) {
String prefix;
HType type = instruction.instructionType;
- if (type.isMutableArray(compiler)) {
+ if (type.isExtendableArray(compiler)) {
+ prefix = 'e';
+ } else if (type.isFixedArray(compiler)) {
+ prefix = 'f';
+ } else if (type.isMutableArray(compiler)) {
prefix = 'm';
} else if (type.isReadableArray(compiler)) {
prefix = 'a';
- } else if (type.isExtendableArray(compiler)) {
- prefix = 'e';
+ } else if (type.isString(compiler)) {
+ prefix = 's';
+ } else if (type.isIndexable(compiler)) {
+ prefix = 'r';
} else if (type == HType.BOOLEAN) {
prefix = 'b';
} else if (type == HType.INTEGER) {
@@ -193,14 +199,10 @@
prefix = 'd';
} else if (type == HType.NUMBER) {
prefix = 'n';
- } else if (type.isString(compiler)) {
- prefix = 's';
} else if (type == HType.UNKNOWN) {
prefix = 'v';
} else if (type == HType.CONFLICTING) {
prefix = 'c';
- } else if (type.isIndexable(compiler)) {
- prefix = 'r';
} else if (type == HType.NULL) {
prefix = 'u';
} else {

Powered by Google App Engine
This is Rietveld 408576698