Merge pull request #9169 from dotty-staging/fix-#9168

Fix #9168: Use `()` for `finally` with empty expression

Author: nicolasstucki

compiler/src/dotty/tools/dotc/parsing/Parsers.scala

@@ -2004,7 +2004,12 @@ object Parsers {
           }
 
           val finalizer =
-            if (in.token == FINALLY) { in.nextToken(); subExpr() }
+            if (in.token == FINALLY) {
+              in.nextToken();
+              val expr = subExpr()
+              if expr.span.exists then expr
+              else Literal(Constant(())) // finally without an expression
+            }
             else {
               if (handler.isEmpty) warning(
                 EmptyCatchAndFinallyBlock(body),

tests/neg-custom-args/fatal-warnings/i9168.scala

@@ -0,0 +1 @@
+def g: Int = try 42 finally ; // error