Skip to content

Commit dd1d712

Browse files
committed
Added "runsAfter" requirements.
1 parent 065b22d commit dd1d712

File tree

3 files changed

+6
-2
lines changed

3 files changed

+6
-2
lines changed

src/dotty/tools/dotc/transform/Erasure.scala

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ class Erasure extends Phase with DenotTransformer {
2828
override def name: String = "erasure"
2929

3030
/** List of names of phases that should precede this phase */
31-
override def runsAfter: Set[String] = Set("typeTestsCasts", "intercepted", "splitter")
31+
override def runsAfter: Set[String] = Set("typeTestsCasts", "intercepted", "splitter", "nullarify")
3232

3333
def transform(ref: SingleDenotation)(implicit ctx: Context): SingleDenotation = ref match {
3434
case ref: SymDenotation =>

src/dotty/tools/dotc/transform/LazyVals.scala

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ class LazyValTranformContext {
6868
override def runsAfterGroupsOf: Set[String] = Set("lazyValsModules")
6969
/** List of names of phases that should have finished their processing of all compilation units
7070
* before this phase starts */
71-
override def runsAfter: Set[String] = Set("lazyValsModules")
71+
override def runsAfter: Set[String] = Set("lazyValsModules") // @darkdimius why repeat; it's already in runsAfterGroupsOf?
7272

7373
/** List of names of phases that should have finished processing of tree
7474
* before this phase starts processing same tree */

src/dotty/tools/dotc/transform/Nullarify.scala

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,10 @@ class Nullarify extends TreeTransform with InfoTransformer {
3939

4040
override def name: String = "nullarify"
4141

42+
override def runsAfterGroupsOf: Set[String] = Set("splitter")
43+
// assumes idents and selects have symbols; interferes with splitter distribution
44+
// that's why it's "after group".
45+
4246
override def transformApply(tree: Apply)(implicit ctx: Context, info: TransformerInfo): Tree =
4347
ctx.traceIndented(s"transforming ${tree.show} at phase ${ctx.phase}", show = true) {
4448

0 commit comments

Comments
 (0)