diff options
| author | Zalathar <Zalathar@users.noreply.github.com> | 2024-10-24 22:23:13 +1100 |
|---|---|---|
| committer | Zalathar <Zalathar@users.noreply.github.com> | 2024-10-25 12:42:42 +1100 |
| commit | 0a96176533ad5c0f8dc3b42c64bf907fb9c27f0b (patch) | |
| tree | 2f48eabbac0a91763e5e5b26f08042944e051d2c /compiler/rustc_codegen_llvm/src/coverageinfo/mapgen.rs | |
| parent | 9f8a6be22188b1ac666bae83fc6aaf7f84ba4522 (diff) | |
| download | rust-0a96176533ad5c0f8dc3b42c64bf907fb9c27f0b.tar.gz rust-0a96176533ad5c0f8dc3b42c64bf907fb9c27f0b.zip | |
coverage: Make obtaining the codegen coverage context infallible
In all the situations where this context is needed, it should always be available.
Diffstat (limited to 'compiler/rustc_codegen_llvm/src/coverageinfo/mapgen.rs')
| -rw-r--r-- | compiler/rustc_codegen_llvm/src/coverageinfo/mapgen.rs | 12 |
1 files changed, 2 insertions, 10 deletions
diff --git a/compiler/rustc_codegen_llvm/src/coverageinfo/mapgen.rs b/compiler/rustc_codegen_llvm/src/coverageinfo/mapgen.rs index b69ca623413..7a2a802eb4e 100644 --- a/compiler/rustc_codegen_llvm/src/coverageinfo/mapgen.rs +++ b/compiler/rustc_codegen_llvm/src/coverageinfo/mapgen.rs @@ -54,11 +54,7 @@ pub(crate) fn finalize(cx: &CodegenCx<'_, '_>) { add_unused_functions(cx); } - let function_coverage_map = match cx.coverage_context() { - Some(ctx) => ctx.take_function_coverage_map(), - None => return, - }; - + let function_coverage_map = cx.coverage_cx().take_function_coverage_map(); if function_coverage_map.is_empty() { // This module has no functions with coverage instrumentation return; @@ -543,9 +539,5 @@ fn add_unused_function_coverage<'tcx>( // zero, because none of its counters/expressions are marked as seen. let function_coverage = FunctionCoverageCollector::unused(instance, function_coverage_info); - if let Some(coverage_context) = cx.coverage_context() { - coverage_context.function_coverage_map.borrow_mut().insert(instance, function_coverage); - } else { - bug!("Could not get the `coverage_context`"); - } + cx.coverage_cx().function_coverage_map.borrow_mut().insert(instance, function_coverage); } |
