diff options
| author | Zalathar <Zalathar@users.noreply.github.com> | 2023-10-15 13:13:05 +1100 |
|---|---|---|
| committer | Zalathar <Zalathar@users.noreply.github.com> | 2023-10-16 21:05:45 +1100 |
| commit | 9bb27f3adfe1df21d1c85b96aaf75f0b349d3ab4 (patch) | |
| tree | 98b6f0420413ff7ffcbb80eaf33448df508e6d38 /compiler/rustc_mir_transform/src/coverage | |
| parent | 7bbe4be5685c7e3ba7bb72f921cee08f48db429d (diff) | |
| download | rust-9bb27f3adfe1df21d1c85b96aaf75f0b349d3ab4.tar.gz rust-9bb27f3adfe1df21d1c85b96aaf75f0b349d3ab4.zip | |
coverage: Remove redundant field `prev_expn_span`
This span can always be retrieved from `prev`, so there is no need to store it separately.
Diffstat (limited to 'compiler/rustc_mir_transform/src/coverage')
| -rw-r--r-- | compiler/rustc_mir_transform/src/coverage/spans.rs | 9 |
1 files changed, 2 insertions, 7 deletions
diff --git a/compiler/rustc_mir_transform/src/coverage/spans.rs b/compiler/rustc_mir_transform/src/coverage/spans.rs index 2e415a61417..3e627ac7a09 100644 --- a/compiler/rustc_mir_transform/src/coverage/spans.rs +++ b/compiler/rustc_mir_transform/src/coverage/spans.rs @@ -215,9 +215,6 @@ struct CoverageSpansGenerator<'a> { /// is mutated. prev_original_span: Span, - /// A copy of the expn_span from the prior iteration. - prev_expn_span: Option<Span>, - /// One or more `CoverageSpan`s with the same `Span` but different `BasicCoverageBlock`s, and /// no `BasicCoverageBlock` in this list dominates another `BasicCoverageBlock` in the list. /// If a new `curr` span also fits this criteria (compared to an existing list of @@ -276,7 +273,6 @@ impl<'a> CoverageSpansGenerator<'a> { curr_original_span: DUMMY_SP, some_prev: None, prev_original_span: DUMMY_SP, - prev_expn_span: None, pending_dups: Vec::new(), refined_spans: Vec::with_capacity(basic_coverage_blocks.num_nodes() * 2), }; @@ -399,8 +395,8 @@ impl<'a> CoverageSpansGenerator<'a> { /// span that ends just after the macro name and its subsequent `!`. fn maybe_push_macro_name_span(&mut self) { let Some(visible_macro) = self.curr().visible_macro(self.body_span) else { return }; - if let Some(prev_expn_span) = &self.prev_expn_span - && prev_expn_span.ctxt() == self.curr().expn_span.ctxt() + if let Some(prev) = &self.some_prev + && prev.expn_span.ctxt() == self.curr().expn_span.ctxt() { return; } @@ -479,7 +475,6 @@ impl<'a> CoverageSpansGenerator<'a> { /// Advance `prev` to `curr` (if any), and `curr` to the next `CoverageSpan` in sorted order. fn next_coverage_span(&mut self) -> bool { if let Some(curr) = self.some_curr.take() { - self.prev_expn_span = Some(curr.expn_span); self.some_prev = Some(curr); self.prev_original_span = self.curr_original_span; } |
