Skip to content
This repository has been archived by the owner on Nov 29, 2020. It is now read-only.

Instance instantiation in the outermost scope #38

Open
hobwekiva opened this issue Jul 15, 2018 · 0 comments
Open

Instance instantiation in the outermost scope #38

hobwekiva opened this issue Jul 15, 2018 · 0 comments
Labels
enhancement New feature or request good first issue Good for newcomers optimization Code optimization opportunity.

Comments

@hobwekiva
Copy link
Contributor

For future docs:

Instance instantiation in the outermost scope

Consider the following snippet:

implicit def eqList[A](implicit A: Eq[A]): Eq[List[A]] =
  new Eq[A] { ... }
implicit class EqOps[A](value: A) {
  def ===(other: A)(implicit A: Eq[A]): Boolean = A.equal(value, other)
}
...
object Test {
  def test: Unit = {
    val a : List[Int] = ...
    val b : List[Int] = ...

    for (i < 0 until 100) {
      a === b
    }
  }
}

The body of the loop will compile to:

new EqOps(a).===(b)(eqList(eqInt))

So there are two allocations per comparison, one allocation of the syntax extension class and one for the instance of Eq[List[A]]. This optimization should fix the latter by instantiating the instance in the outermost static scope,

object Test {
  implicit val $ev1: Eq[List[A]] = eqList(eqInt)

  def test: Unit = {
    val a : List[Int] = ...
    val b : List[Int] = ...

    for (i < 0 until 100) {
      new EqOps(a).===(b)($ev1)
    }
  }
}

or at the beginning of the function:

class Test {
  def test: Unit = {
    implicit val $ev1: Eq[List[A]] = eqList(eqInt)
    val a : List[Int] = ...
    val b : List[Int] = ...

    for (i < 0 until 100) {
      new EqOps(a).===(b)($ev1)
    }
  }
}

Requirements

  • No side-effects in implicits.
@hobwekiva hobwekiva added enhancement New feature or request good first issue Good for newcomers optimization Code optimization opportunity. labels Jul 18, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request good first issue Good for newcomers optimization Code optimization opportunity.
Projects
None yet
Development

No branches or pull requests

1 participant