Make HBreak and HContinue not extend HGoto. They now extend HJump which is independent of HGoto. To…

lib/compiler/implementation/ssa/codegen.dart

Index: lib/compiler/implementation/ssa/codegen.dart
diff --git a/lib/compiler/implementation/ssa/codegen.dart b/lib/compiler/implementation/ssa/codegen.dart
index 4c944d651fcf9b1a3b81feedcdea3b2bece48ab8..64683fcb1c306b68f0b528d42a312aef173c2fbd 100644
--- a/lib/compiler/implementation/ssa/codegen.dart
+++ b/lib/compiler/implementation/ssa/codegen.dart
@@ -893,7 +893,7 @@ class SsaCodeGenerator implements HVisitor, HBlockInformationVisitor {
   }
 
   void emitAssignment(String destination, String source) {
-    if (isGeneratingExpression()) { 
+    if (isGeneratingExpression()) {
       addExpressionSeparator();
     } else {
       addIndentation();
@@ -1238,9 +1238,7 @@ class SsaCodeGenerator implements HVisitor, HBlockInformationVisitor {
 
   bool isEmptyElse(HBasicBlock start, HBasicBlock end) {
     if (start !== end) return false;
-    if (start.last is !HGoto
-        || start.last is HBreak
-        || start.last is HContinue) {
+    if (start.last is !HGoto) {
       return false;

[ngeoffray, 2012/06/06 11:05:29]

one line?

[Lasse Reichstein Nielsen, 2012/06/06 13:04:07]

NEVER!
Ahem.

I'd actually rather combine it with the test above, and keep the block. 
I don't like hiding early returns at the end of innocuously looking lines.

     }
     for (HInstruction instruction = start.first;