Merge pull request #4571 from dotty-staging/fix-#4492

Fix #4492: Load module instance instead of calling static methods

Author: nicolasstucki

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

@@ -10,6 +10,7 @@ import dotty.tools.dotc.core.Decorators._
 import dotty.tools.dotc.core.Flags.Package
 import dotty.tools.dotc.core.NameKinds.FlatName
 import dotty.tools.dotc.core.Names.Name
+import dotty.tools.dotc.core.StdNames.str.MODULE_INSTANCE_FIELD
 import dotty.tools.dotc.core.quoted._
 import dotty.tools.dotc.core.Types._
 import dotty.tools.dotc.core.Symbols._
@@ -130,7 +131,9 @@ object Splicer {
     private def loadModule(sym: Symbol): (Class[_], Object) = {
       if (sym.owner.is(Package)) {
         // is top level object
-        (loadClass(sym.companionModule.fullName), null)
+        val moduleClass = loadClass(sym.fullName)
+        val moduleInstance = moduleClass.getField(MODULE_INSTANCE_FIELD).get(null)
+        (moduleClass, moduleInstance)
       } else {
         // nested object in an object
         val clazz = loadClass(sym.fullNameSeparated(FlatName))

tests/run/i4492.check

@@ -0,0 +1 @@
+Ok

tests/run/i4492/quoted_1.scala

@@ -0,0 +1,6 @@
+
+trait Index
+
+object Index {
+  inline def succ(prev: Index): Unit = ~{ '(println("Ok")) }
+}

tests/run/i4492/quoted_2.scala

@@ -0,0 +1,6 @@
+
+object Test {
+  def main(args: Array[String]): Unit = {
+    Index.succ(null)
+  }
+}