Re: [RFC PATCH 16/20] coverage: add MC/DC measurement

[Sasha Levin]

On Wed, Oct 28, 2026 at 12:14:32PM -0700, Nathan Chancellor wrote:
>On Wed, Oct 28, 2026 at 07:59:40AM -0400, Sasha Levin wrote:
>> Larger decisions, and decisions the compiler can't instrument, produce
>> warnings. Starting with Clang 22 they are kept from becoming errors
>> with CONFIG_WERROR; older versions can't be used together with it.
>
>What is the reason for keeping these warnings in the build as opposed to
>just disabling them altogether? Are developers expected to address them
>and if so, how? It seems odd to keep them in the build but prevent them
>from breaking the build unless they are not intended to be acted on, in
>which case they should just be disabled outright.

They're not meant to be acted on. Clang only put them in a warning group
(-Wpgo-coverage) in 22, so before that there was no way to turn them
off, which is why I just left them on. I'll disable them in v2 where
Clang allows it.

-- 
Thanks,
Sasha