Make sure that we don't assign phis before its uses.

tests/language/loop_exchange3_test.dart

Index: tests/language/loop_exchange3_test.dart
diff --git a/tests/language/loop_exchange3_test.dart b/tests/language/loop_exchange3_test.dart
new file mode 100644
index 0000000000000000000000000000000000000000..defba59ba2865ff8942acd334d0332c77d291f13
--- /dev/null
+++ b/tests/language/loop_exchange3_test.dart
@@ -0,0 +1,13 @@
+// Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
+// for details. All rights reserved. Use of this source code is governed by a
+// BSD-style license that can be found in the LICENSE file.
+
+// This program tripped dart2js.

[ngeoffray, 2012/05/03 07:29:14]

This program works for me?

[floitsch, 2012/05/03 09:11:29]

I failed this test when I had implemented a bad version of this CL. So yes: no
checked in version of dart2js ever failed this test. But a local one did. I can
remove the comment, though.

+main() {
+  int foo;
+  for (var i = 0; i < 10; foo = i, i++) {
+    if(i > 0) { // e is not evaluated before the first iteration

[ngeoffray, 2012/05/03 07:29:14]

e?

[floitsch, 2012/05/03 09:11:29]

removed.

+      Expect.equals(i - 1, foo);
+    }
+  }
+}