Move ExpandSAMs to miniphase group before HoistSuperArgs

There are interactions for treating partial functions that require
a split into different groups.

This move uncovered two follow-on errors which were previously not detected.

Author: odersky

compiler/src/dotty/tools/dotc/Compiler.scala

@@ -61,9 +61,9 @@ class Compiler {
          new CookComments,           // Cook the comments: expand variables, doc, etc.
          new CheckStatic,            // Check restrictions that apply to @static members
          new BetaReduce,             // Reduce closure applications
+         new ExpandSAMs,             // Expand single abstract method closures to anonymous classes
          new init.Checker) ::        // Check initialization of objects
     List(new ElimRepeated,           // Rewrite vararg parameters and arguments
-         new ExpandSAMs,             // Expand single abstract method closures to anonymous classes
          new ProtectedAccessors,     // Add accessors for protected members
          new ExtensionMethods,       // Expand methods of value classes with extension methods
          new UncacheGivenAliases,    // Avoid caching RHS of simple parameterless given aliases

compiler/src/dotty/tools/dotc/transform/ByNameClosures.scala

@@ -26,6 +26,10 @@ class ByNameClosures extends TransformByNameApply with IdentityDenotTransformer
 
   override def phaseName: String = ByNameClosures.name
 
+  override def runsAfterGroupsOf: Set[String] = Set(ExpandSAMs.name)
+    // ExpanSAMs applied to partial functions creates methods that need
+    // to be fully defined before converting. Test case is pos/i9391.scala.
+
   override def mkByNameClosure(arg: Tree, argType: Type)(using Context): Tree = {
     val meth = newSymbol(
       ctx.owner, nme.ANON_FUN, Synthetic | Method, MethodType(Nil, Nil, argType))

tests/pos/i9391.scala

@@ -0,0 +1,3 @@
+def g(x: => Any): Any = x
+
+val a: PartialFunction[Any => Any, Any] = (f => g(f(0)) match { case v => v })    // was an error, now OK