remove scala.runtime.EnumValues

[bishabosha]

generates an array of the constants immediately after the last enum case

[bishabosha]

so it seems that TASTy does not distinguish between SeqLiteral and JavaSeqLiteral so that is unusable
@smarter @sjrd

[sjrd]

That is disappointing. Is it intended? If yes, I'm afraid you'll have to revert to the the code that generates the Apply.apply(...) call. But in that case, I wonder how TASTy would persist a call to a Java varargs method ... Maybe there's a deeper problem here.

[bishabosha]

@sjrd looks like phase elimRepeated uses expected type of a java varargs method to transform SeqLiteral to JavaSeqLiteral

[smarter]

I think it would make sense to support pickling/unpickling JavaSeqLiteral.

[bishabosha]

@smarter @sjrd not sure what to do here for this example:

class Outer {
  val thunk = { () =>
    enum E { case A }
    E.A
  }
}

@main def Test =
  assert(Outer().thunk().toString == "A")

if I create the $values array with Ident("A"), then this code works, but causes the init checker to complain about access to A, but if I use Select(companionRef, "A") then the init checker does not complain, but then accessing the companion object through selection causes a runtime stack overflow in constructor of E

[sjrd]

I guess you could make it a lazy val to break the cycle in the constructor of E?

[bishabosha]

I'll make it lazy when the owner of the enum is not static

[bishabosha]

this works and can be merged, but should be updated in the future based on how #9664 is resolved