Hi Heinrich,

On Wed, 31 Dec 2025 at 21:11, Heinrich Schuchardt
<[email protected]> wrote:
>
> Since commit 5f520875bdf0 ("kbuild: Bump the build system to 5.1")
> CONFIG_CC_OPTIMIZE_FOR_DEBUG has no effect on the non-host code.
>
> This patch reestablishes the prior logic to add
>
>     -Og -Wno-maybe-uninitialized
>
> to KBUILD_CFLAGS.
>
> Fixes: 5f520875bdf0 ("kbuild: Bump the build system to 5.1")
> Signed-off-by: Heinrich Schuchardt <[email protected]>
> ---
>  Makefile | 5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/Makefile b/Makefile
> index 6b5c5ee019e..2a972af4906 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -869,6 +869,11 @@ endif
>
>  ifdef CONFIG_CC_OPTIMIZE_FOR_SIZE
>  KBUILD_CFLAGS  += -Os
> +else ifdef CONFIG_CC_OPTIMIZE_FOR_DEBUG
> +-KBUILD_CFLAGS  += -Og
> +# Avoid false positives -Wmaybe-uninitialized
> +# https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78394
> +KBUILD_CFLAGS  += -Wno-maybe-uninitialized
>  else

There's a ifdev CONFIG_CC_OPTIMIZE_FOR_DEBUG a few lines above.
Wouldn't it be better to move the HOSTCFLAGS under the new ifdef as
well?

Cheers
/Ilias
>  KBUILD_CFLAGS   += -O2
>  endif
> --
> 2.51.0
>

Reply via email to