Merge pull request #3240 from dotty-staging/drop-no-effects-drop-blocks

OlivierBlanvillain · web-flow · commit 20425bb0aa20 · 2017-10-05T09:21:48.000+02:00
Drop block early in DropNoEffects
diff --git a/compiler/src/dotty/tools/dotc/transform/localopt/DropNoEffects.scala b/compiler/src/dotty/tools/dotc/transform/localopt/DropNoEffects.scala
@@ -142,7 +142,7 @@ class DropNoEffects(val simplifyPhase: Simplify) extends Optimisation {
         case _ =>
           rec :: args.map(keepOnlySideEffects)
       }
-      Block(prefix, unitLiteral)
+      seq(prefix, unitLiteral)
 
     case t => t
   }

Original file line number	Diff line number	Diff line change
`@@ -142,7 +142,7 @@ class DropNoEffects(val simplifyPhase: Simplify) extends Optimisation {`
`142`	`142`	`case _ =>`
`143`	`143`	`rec :: args.map(keepOnlySideEffects)`
`144`	`144`	`}`
`145`		`- Block(prefix, unitLiteral)`
	`145`	`+ seq(prefix, unitLiteral)`
`146`	`146`
`147`	`147`	`case t => t`
`148`	`148`	`}`