On 08.04.2025 17:38, Volodymyr Babchuk wrote: > Jan Beulich <jbeul...@suse.com> writes: >> On 07.04.2025 18:46, Volodymyr Babchuk wrote: >>> @@ -133,19 +134,18 @@ $(filter %.init.o,$(obj-y) $(obj-bin-y) $(extra-y)): >>> CFLAGS-y += -DINIT_SECTIONS >>> >>> non-init-objects = $(filter-out %.init.o, $(obj-y) $(obj-bin-y) $(extra-y)) >>> >>> -ifeq ($(CONFIG_COVERAGE),y) >>> ifeq ($(CONFIG_CC_IS_CLANG),y) >>> - COV_FLAGS := -fprofile-instr-generate -fcoverage-mapping >>> + cov-flags-$(CONFIG_COVERAGE) := -fprofile-instr-generate >>> -fcoverage-mapping >>> else >>> - COV_FLAGS := -fprofile-arcs -ftest-coverage >>> + cov-flags-$(CONFIG_COVERAGE) := -fprofile-arcs -ftest-coverage >>> + cov-flags-$(CONFIG_CONDITION_COVERAGE) += -fcondition-coverage >> >> Why's this inside the remaining ifeq(,)? Surely there's at least a chance for >> Clang to also support the option at some point? > > Yes, but Clang uses different option: -fcoverage-mcdc. I see no sense in > adding it right now, as Xen does not support version 10 of llvm > profiling format, in which they added MC/DC support.
Okay, but then can you amend "Clang is not supported right now" in the description by another half sentence clarifying why that is? Jan
