Skip to content

Commit 58e34e8

Browse files
committed
Fix bug in phase identification for Ycheck
-Ycheck:era checked after phase resolveSuper. This was due to an overly simplistic containsPhase check.
1 parent f43cffc commit 58e34e8

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/dotty/tools/dotc/core/Phases.scala

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ import dotty.tools.backend.jvm.{LabelDefs, GenBCode}
77
import util.DotClass
88
import DenotTransformers._
99
import Denotations._
10+
import Decorators._
1011
import config.Printers._
1112
import scala.collection.mutable.{ListBuffer, ArrayBuffer}
1213
import dotty.tools.dotc.transform.TreeTransforms.{TreeTransformer, MiniPhase, TreeTransform}
@@ -121,10 +122,9 @@ object Phases {
121122
phase
122123
}
123124
squashedPhases += phaseToAdd
124-
val shouldAddYCheck = YCheckAfter.exists(nm => phaseToAdd.phaseName.contains(nm)) || YCheckAll
125+
val shouldAddYCheck = YCheckAfter.containsPhase(phaseToAdd) || YCheckAll
125126
if (shouldAddYCheck) {
126127
val checker = new TreeChecker
127-
128128
squashedPhases += checker
129129
}
130130
}

0 commit comments

Comments
 (0)