dart2js cps: Scalar replacement of aggregates

pkg/compiler/lib/src/js_backend/codegen/task.dart

Index: pkg/compiler/lib/src/js_backend/codegen/task.dart
diff --git a/pkg/compiler/lib/src/js_backend/codegen/task.dart b/pkg/compiler/lib/src/js_backend/codegen/task.dart
index 00cd5b9d90eb3002b1b4fa9ba9c5aad40d4e52d7..16c882ac3274d7786dbf57624bdc633ae3dfdaee 100644
--- a/pkg/compiler/lib/src/js_backend/codegen/task.dart
+++ b/pkg/compiler/lib/src/js_backend/codegen/task.dart
@@ -175,6 +175,7 @@ class CpsFunctionCompiler implements FunctionCompiler {
     dumpTypedIR(cpsNode, typePropagator);
     applyCpsPass(new LoopInvariantCodeMotion());
     applyCpsPass(new ShareInterceptors());
+    applyCpsPass(new ScalarReplacer(compiler));

[asgerf, 2015/08/31 09:57:30]

We should rethink the order of these passes.

ScalarReplacer and MutableVariableEliminator should be before
LoopInvariantCodeMotion and ShareInterceptors, so a call like
getInterceptor(box.x) can be shared/hoisted if the box is eliminated.

We can change the order in another CL though.

[sra1, 2015/09/02 01:12:57]

Acknowledged.

     applyCpsPass(new ShrinkingReducer());
     applyCpsPass(new MutableVariableEliminator());
     applyCpsPass(new RedundantJoinEliminator());