Revert getClass changes

Move getClass back to Any. This is needed to make zio pass.

Author: odersky

community-build/test/scala/dotty/communitybuild/CommunityBuildTest.scala

@@ -101,7 +101,7 @@ class CommunityBuildTestA extends CommunityBuildTest:
   @Test def ujson = projects.ujson.run()
   @Test def upickle = projects.upickle.run()
   @Test def utest = projects.utest.run()
-  //@Test def zio = projects.zio.run()  // fails with some runtime failures of non-advancing fibers
+  @Test def zio = projects.zio.run() 
 end CommunityBuildTestA
 
 @Category(Array(classOf[TestCategory]))

compiler/src/dotty/tools/backend/jvm/BCodeSkelBuilder.scala

@@ -716,7 +716,7 @@ trait BCodeSkelBuilder extends BCodeHelpers {
       val rhs = dd.rhs
       val vparamss = dd.vparamss
       // the only method whose implementation is not emitted: getClass()
-      if (dd.symbol eq defn.Matchable_getClass) { return }
+      if (dd.symbol eq defn.Any_getClass) { return }
       assert(mnode == null, "GenBCode detected nested method.")
 
       methSymbol  = dd.symbol

compiler/src/dotty/tools/backend/sjs/JSCodeGen.scala

@@ -1731,7 +1731,7 @@ class JSCodeGen()(using genCtx: Context) {
     val Apply(fun @ Select(sup @ Super(qual, _), _), args) = tree
     val sym = fun.symbol
 
-    if (sym == defn.Matchable_getClass) {
+    if (sym == defn.Any_getClass) {
       // The only primitive that is also callable as super call
       js.GetClass(genThis())
     } else if (currentClassSym.isNonNativeJSClass) {

compiler/src/dotty/tools/dotc/core/Definitions.scala

@@ -295,17 +295,17 @@ class Definitions {
     @tu lazy val Matchable_isInstanceOf: TermSymbol = enterT1ParameterlessMethod(MatchableClass, nme.isInstanceOf_, _ => BooleanType, Final)
 
     /** def getClass[A >: this.type](): Class[? <: A] */
-    @tu lazy val Matchable_getClass: TermSymbol =
+    @tu lazy val Any_getClass: TermSymbol =
         enterPolyMethod(
-          MatchableClass, nme.getClass_, 1,
+          AnyClass, nme.getClass_, 1,
           pt => MethodType(Nil, ClassClass.typeRef.appliedTo(TypeBounds.upper(pt.paramRefs(0)))),
           Final,
-          bounds = TypeBounds.lower(MatchableClass.thisType))
+          bounds = TypeBounds.lower(AnyClass.thisType))
 
-    def AnyMethods: List[TermSymbol] = List(Any_==, Any_!=, Any_equals, Any_hashCode,
+    def AnyMethods: List[TermSymbol] = List(Any_==, Any_!=, Any_equals, Any_hashCode, Any_getClass,
       Any_toString, Any_##, Matchable_isInstanceOf, Any_asInstanceOf, Any_typeTest, Any_typeCast)
 
-    def MatchableMethods: List[TermSymbol] = List(Matchable_getClass, Matchable_isInstanceOf)
+    def MatchableMethods: List[TermSymbol] = List(Matchable_isInstanceOf)
 
     @tu lazy val MatchableMethodNames = MatchableMethods.map(_.name)
 
@@ -443,7 +443,7 @@ class Definitions {
 
   /** Methods in Object, Matchable, and Any that do not have a side effect */
   @tu lazy val pureMethods: List[TermSymbol] = List(Any_==, Any_!=, Any_equals, Any_hashCode,
-    Any_toString, Any_##, Matchable_getClass, Matchable_isInstanceOf, Any_typeTest, Object_eq, Object_ne)
+    Any_toString, Any_##, Any_getClass, Matchable_isInstanceOf, Any_typeTest, Object_eq, Object_ne)
 
   @tu lazy val AnyKindClass: ClassSymbol = {
     val cls = newCompleteClassSymbol(ScalaPackageClass, tpnme.AnyKind, AbstractFinal | Permanent, Nil)

compiler/src/dotty/tools/dotc/transform/init/Env.scala

@@ -25,7 +25,7 @@ case class Env(ctx: Context) {
 
   // Methods that should be ignored in the checking
   lazy val ignoredMethods: Set[Symbol] = Set(
-    defn.Matchable_getClass,
+    defn.Any_getClass,
     defn.Matchable_isInstanceOf,
     defn.Object_eq,
     defn.Object_ne,

compiler/src/dotty/tools/dotc/typer/RefChecks.scala

@@ -864,7 +864,7 @@ object RefChecks {
             // getClass in primitive value classes is defined in the standard library as:
             //     override def getClass(): Class[Int] = ???
             // However, it's not actually an override in Dotty because our Any#getClass
-            // is polymorphic (see `Definitions#Matchable_getClass`), so since we can't change
+            // is polymorphic (see `Definitions#Any_getClass`), so since we can't change
             // the standard library, we need to drop the override flag without reporting
             // an error.
             if (!(member.name == nme.getClass_ && clazz.isPrimitiveValueClass))