From 63107e02c2f78dd9c013c8f6428b55101bcd38d2 Mon Sep 17 00:00:00 2001 From: Roger Pau Monne Date: Tue, 25 Nov 2025 15:23:34 +0100 Subject: [PATCH] xen/linker: be more selective about defining LLVM_COV_* MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit There's no need to place the LLVM coverage sections in the linker script if not building with clang, or if code coverage is not enabled. No functional change intended. Fixes: 186c02e85961 ('xen: Support LLVM raw profile versions 5, 6, 7, 8, 9, and 10') Reported-by: Andrew Cooper Signed-off-by: Roger Pau Monné Reviewed-by: Jan Beulich --- xen/include/xen/xen.lds.h | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/xen/include/xen/xen.lds.h b/xen/include/xen/xen.lds.h index d80c895959..f54fb2d152 100644 --- a/xen/include/xen/xen.lds.h +++ b/xen/include/xen/xen.lds.h @@ -81,6 +81,7 @@ .stab.index 0 : { *(.stab.index) } \ .stab.indexstr 0 : { *(.stab.indexstr) } +#if defined(CONFIG_CC_IS_CLANG) && defined(CONFIG_COVERAGE) /* Clang coverage sections. */ #define LLVM_COV_RW_DATA \ DECL_SECTION(__llvm_prf_cnts) { *(__llvm_prf_cnts) } \ @@ -93,6 +94,11 @@ #define LLVM_COV_DEBUG \ DECL_DEBUG(__llvm_covfun, 8) \ DECL_DEBUG(__llvm_covmap, 8) +#else +#define LLVM_COV_RW_DATA +#define LLVM_COV_RO_DATA +#define LLVM_COV_DEBUG +#endif /* * ELF sections.