Skip to content

Commit e2893f7

Browse files
committed
Auto merge of #149757 - jdonszelmann:revert-duplicate-span-lowering, r=nnethercote
Revert "early return on duplicate span lowerings" r? `@nnethercote` reverts #149060 because of perf regressions that are still wild to me
2 parents d5525a7 + 942a08b commit e2893f7

File tree

1 file changed

+0
-7
lines changed
  • compiler/rustc_ast_lowering/src

1 file changed

+0
-7
lines changed

compiler/rustc_ast_lowering/src/lib.rs

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -225,13 +225,6 @@ struct SpanLowerer {
225225
impl SpanLowerer {
226226
fn lower(&self, span: Span) -> Span {
227227
if self.is_incremental {
228-
// early return: span lowering takes some time since it accesses the query dependency graph
229-
// to make sure we rerun the right code paths when spans change. When we've already lowered a span,
230-
// or don't have to, bail out ASAP.
231-
if span.is_dummy() || span.parent().is_some_and(|i| i == self.def_id) {
232-
return span;
233-
}
234-
235228
span.with_parent(Some(self.def_id))
236229
} else {
237230
// Do not make spans relative when not using incremental compilation.

0 commit comments

Comments
 (0)