bjoernager/rust
bjoernager/rust
1
Fork 0
Code Activity

Disable coverage instrumentation for naked functions

Browse source
This commit is contained in:
Tomasz Miąsko 2022-12-02 00:00:00 +00:00
parent 32e613bbaa
commit c955add18c
2 changed files with 23 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

4
compiler/rustc_hir_analysis/src/collect.rs
View file

@ -2073,6 +2073,10 @@ fn codegen_fn_attrs(tcx: TyCtxt<'_>, did: DefId) -> CodegenFnAttrs {
} }
} }
if codegen_fn_attrs.flags.contains(CodegenFnAttrFlags::NAKED) {
codegen_fn_attrs.flags |= CodegenFnAttrFlags::NO_COVERAGE;
}
// Weak lang items have the same semantics as "std internal" symbols in the // Weak lang items have the same semantics as "std internal" symbols in the
// sense that they're preserved through all our LTO passes and only // sense that they're preserved through all our LTO passes and only
// strippable by the linker. // strippable by the linker.

19
src/test/codegen/naked-nocoverage.rs Normal file
View file

@ -0,0 +1,19 @@
// Checks that naked functions are not instrumented by -Cinstrument-coverage.
// Regression test for issue #105170.
//
// needs-asm-support
// needs-profiler-support
// compile-flags: -Cinstrument-coverage
#![crate_type = "lib"]
#![feature(naked_functions)]
use std::arch::asm;
#[naked]
#[no_mangle]
pub unsafe extern "C" fn f() {
// CHECK: define void @f()
// CHECK-NEXT: start:
// CHECK-NEXT: call void asm
// CHECK-NEXT: unreachable
asm!("", options(noreturn));
}