Remove DottyBackendInterface.sourcePos

Author: nicolasstucki

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

@@ -663,7 +663,7 @@ trait BCodeBodyBuilder extends BCodeSkelBuilder {
       var elemKind = arr.elementType
       val argsSize = args.length
       if (argsSize > dims) {
-        ctx.error(s"too many arguments for array constructor: found ${args.length} but array has only $dims dimension(s)", sourcePos(app.span))
+        ctx.error(s"too many arguments for array constructor: found ${args.length} but array has only $dims dimension(s)", app.sourcePos)
       }
       if (argsSize < dims) {
         /* In one step:

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

@@ -59,7 +59,7 @@ trait BCodeHelpers extends BCodeIdiomatic with BytecodeWriters {
       outputDirectory
     } catch {
       case ex: Throwable =>
-        ctx.error(s"Couldn't create file for class $cName\n${ex.getMessage}", sourcePos(csym.span))
+        ctx.error(s"Couldn't create file for class $cName\n${ex.getMessage}", csym.sourcePos)
         null
     }
   }

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

@@ -403,7 +403,7 @@ trait BCodeSkelBuilder extends BCodeHelpers {
         val loc = Local(tk, sym.name.mangledString.toString, nxtIdx, sym.is(Flags.Synthetic))
         val existing = slots.put(sym, loc)
         if (existing.isDefined)
-          ctx.error("attempt to create duplicate local var.", sourcePos(sym.span))
+          ctx.error("attempt to create duplicate local var.", sym.sourcePos)
         assert(tk.size > 0, "makeLocal called for a symbol whose type is Unit.")
         nxtIdx += tk.size
         loc
@@ -453,7 +453,7 @@ trait BCodeSkelBuilder extends BCodeHelpers {
     }
     def lineNumber(tree: Tree): Unit = {
       if (!emitLines || !tree.span.exists) return;
-      val nr = sourcePos(tree.span).line + 1
+      val nr = tree.sourcePos.line + 1
       if (nr != lastEmittedLineNr) {
         lastEmittedLineNr = nr
         lastInsn match {
@@ -556,7 +556,7 @@ trait BCodeSkelBuilder extends BCodeHelpers {
 
       if (params.size > MaximumJvmParameters) {
         // SI-7324
-        ctx.error(s"Platform restriction: a parameter list's length cannot exceed $MaximumJvmParameters.", sourcePos(methSymbol.span))
+        ctx.error(s"Platform restriction: a parameter list's length cannot exceed $MaximumJvmParameters.", methSymbol.sourcePos)
         return
       }
 
@@ -585,7 +585,7 @@ trait BCodeSkelBuilder extends BCodeHelpers {
               ctx.error("Concrete method has no definition: " + dd + (
                 if (ctx.settings.Ydebug.value) "(found: " + methSymbol.owner.info.decls.toList.mkString(", ") + ")"
                 else ""),
-                sourcePos(NoSpan)
+                ctx.source.atSpan(NoSpan)
               )
             case _ =>
               bc emitRETURN returnType

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

@@ -75,8 +75,6 @@ class DottyBackendInterface(val outputDirectory: AbstractFile, val superCallsMap
     ctx.error(msg)
     throw new RuntimeException(msg)
   }
-  def sourcePos(pos: Spans.Span)(implicit ctx: Context): util.SourcePosition =
-    ctx.source.atSpan(pos)
 
   val perRunCaches: Caches = new Caches {
     def newAnyRefMap[K <: AnyRef, V](): mutable.AnyRefMap[K, V] = new mutable.AnyRefMap[K, V]()
@@ -347,7 +345,7 @@ class DottyBackendInterface(val outputDirectory: AbstractFile, val superCallsMap
     def _1: Type = field.tpe match {
       case JavaArrayType(elem) => elem
       case _ =>
-        ctx.error(s"JavaSeqArray with type ${field.tpe} reached backend: $field", sourcePos(field.span))
+        ctx.error(s"JavaSeqArray with type ${field.tpe} reached backend: $field", field.sourcePos)
         UnspecifiedErrorType
     }
     def _2: List[Tree] = field.elems