nordic-dev.net/rust
nordic-dev.net/rust
2
0
Fork 0
mirror of https://github.com/rust-lang/rust.git synced 2024-11-26 08:44:35 +00:00
Code Issues Packages Projects Releases Wiki Activity

coverage: Clean up maybe_push_macro_name_span

Browse Source
This commit is contained in:
Zalathar 2023-12-04 19:43:44 +11:00
parent 9358642e3b
commit ff3af59f2b
1 changed files with 10 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
compiler/rustc_mir_transform/src/coverage/spans.rs
View File

@ -379,18 +379,22 @@ impl<'a> CoverageSpansGenerator<'a> {
return; return;
} }
let merged_prefix_len = self.curr_original_span.lo() - curr.span.lo(); // The split point is relative to `curr_original_span`,
let after_macro_bang = merged_prefix_len + BytePos(visible_macro.as_str().len() as u32 + 1); // because `curr.span` may have been merged with preceding spans.
if self.curr().span.lo() + after_macro_bang > self.curr().span.hi() { let split_point_after_macro_bang = self.curr_original_span.lo()
+ BytePos(visible_macro.as_str().len() as u32)
+ BytePos(1); // add 1 for the `!`
if split_point_after_macro_bang > curr.span.hi() {
// Something is wrong with the macro name span; // Something is wrong with the macro name span;
// return now to avoid emitting malformed mappings. // return now to avoid emitting malformed mappings.
// FIXME(#117788): Track down why this happens. // FIXME(#117788): Track down why this happens.
return; return;
} }
let mut macro_name_cov = curr.clone(); let mut macro_name_cov = curr.clone();
self.curr_mut().span = curr.span.with_lo(curr.span.lo() + after_macro_bang); macro_name_cov.span = macro_name_cov.span.with_hi(split_point_after_macro_bang);
macro_name_cov.span = self.curr_mut().span = curr.span.with_lo(split_point_after_macro_bang);
macro_name_cov.span.with_hi(macro_name_cov.span.lo() + after_macro_bang);
debug!( debug!(
" and curr starts a new macro expansion, so add a new span just for \ " and curr starts a new macro expansion, so add a new span just for \
the macro `{visible_macro}!`, new span={macro_name_cov:?}", the macro `{visible_macro}!`, new span={macro_name_cov:?}",