Skip to content

Commit d111aa8

Browse files
Trivial tweaks to stop tracking source span directly
1 parent 25a615b commit d111aa8

File tree

3 files changed

+20
-16
lines changed

3 files changed

+20
-16
lines changed

Diff for: compiler/rustc_borrowck/src/region_infer/mod.rs

+18-14
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ use rustc_middle::ty::{self, RegionVid, Ty, TyCtxt, TypeFoldable, UniverseIndex,
2222
use rustc_mir_dataflow::points::DenseLocationMap;
2323
use rustc_span::hygiene::DesugaringKind;
2424
use rustc_span::{DUMMY_SP, Span};
25-
use tracing::{debug, instrument, trace};
25+
use tracing::{Level, debug, enabled, instrument, trace};
2626

2727
use crate::BorrowckInferCtxt;
2828
use crate::constraints::graph::{self, NormalConstraintGraph, RegionGraph};
@@ -327,11 +327,13 @@ fn sccs_info<'tcx>(infcx: &BorrowckInferCtxt<'tcx>, sccs: &ConstraintSccs) {
327327
let mut var_to_origin_sorted = var_to_origin.clone().into_iter().collect::<Vec<_>>();
328328
var_to_origin_sorted.sort_by_key(|vto| vto.0);
329329

330-
let mut reg_vars_to_origins_str = "region variables to origins:\n".to_string();
331-
for (reg_var, origin) in var_to_origin_sorted.into_iter() {
332-
reg_vars_to_origins_str.push_str(&format!("{reg_var:?}: {origin:?}\n"));
330+
if enabled!(Level::DEBUG) {
331+
let mut reg_vars_to_origins_str = "region variables to origins:\n".to_string();
332+
for (reg_var, origin) in var_to_origin_sorted.into_iter() {
333+
reg_vars_to_origins_str.push_str(&format!("{reg_var:?}: {origin:?}\n"));
334+
}
335+
debug!("{}", reg_vars_to_origins_str);
333336
}
334-
debug!("{}", reg_vars_to_origins_str);
335337

336338
let num_components = sccs.num_sccs();
337339
let mut components = vec![FxIndexSet::default(); num_components];
@@ -342,16 +344,18 @@ fn sccs_info<'tcx>(infcx: &BorrowckInferCtxt<'tcx>, sccs: &ConstraintSccs) {
342344
components[scc_idx.as_usize()].insert((reg_var, *origin));
343345
}
344346

345-
let mut components_str = "strongly connected components:".to_string();
346-
for (scc_idx, reg_vars_origins) in components.iter().enumerate() {
347-
let regions_info = reg_vars_origins.clone().into_iter().collect::<Vec<_>>();
348-
components_str.push_str(&format!(
349-
"{:?}: {:?},\n)",
350-
ConstraintSccIndex::from_usize(scc_idx),
351-
regions_info,
352-
))
347+
if enabled!(Level::DEBUG) {
348+
let mut components_str = "strongly connected components:".to_string();
349+
for (scc_idx, reg_vars_origins) in components.iter().enumerate() {
350+
let regions_info = reg_vars_origins.clone().into_iter().collect::<Vec<_>>();
351+
components_str.push_str(&format!(
352+
"{:?}: {:?},\n)",
353+
ConstraintSccIndex::from_usize(scc_idx),
354+
regions_info,
355+
))
356+
}
357+
debug!("{}", components_str);
353358
}
354-
debug!("{}", components_str);
355359

356360
// calculate the best representative for each component
357361
let components_representatives = components

Diff for: compiler/rustc_hir_typeck/src/closure.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -970,7 +970,7 @@ impl<'a, 'tcx> FnCtxt<'a, 'tcx> {
970970
self.typeck_results.borrow_mut().user_provided_sigs.insert(expr_def_id, c_result);
971971

972972
// Normalize only after registering in `user_provided_sigs`.
973-
self.normalize(self.tcx.hir_span(hir_id), result)
973+
self.normalize(self.tcx.def_span(expr_def_id), result)
974974
}
975975

976976
/// Invoked when we are translating the coroutine that results

Diff for: compiler/rustc_hir_typeck/src/lib.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,7 @@ fn typeck_with_inspect<'tcx>(
117117

118118
let id = tcx.local_def_id_to_hir_id(def_id);
119119
let node = tcx.hir_node(id);
120-
let span = tcx.hir_span(id);
120+
let span = tcx.def_span(def_id);
121121

122122
// Figure out what primary body this item has.
123123
let body_id = node.body_id().unwrap_or_else(|| {

0 commit comments

Comments
 (0)