[PATCH] malloc: Fix Os build on some ABIs

Collin Funk collin.funk1@gmail.com
Sat Sep 6 22:32:07 GMT 2025


Adhemerval Zanella <adhemerval.zanella@linaro.org> writes:

> I have not checked with all versions for all ABIs, but I saw failures
> with gcc-14 on arm, alpha, hppa, i686, sparc, sh4, and microblaze.
> ---
>  malloc/malloc.c | 6 ++++++
>  1 file changed, 6 insertions(+)
>
> diff --git a/malloc/malloc.c b/malloc/malloc.c
> index 6c02af5a27..1896c55e21 100644
> --- a/malloc/malloc.c
> +++ b/malloc/malloc.c
> @@ -3305,10 +3305,16 @@ tcache_get_align (size_t nb, size_t alignment)
>            mangled = true;
>          }
>  
> +      /* GCC compiling for -Os warns on some architectures that csize maya be
> +	 uninitialized.  However, if 'te' is not NULL, csize is always
> +	 initialized in the loop above.  */
> +      DIAG_PUSH_NEEDS_COMMENT;
> +      DIAG_IGNORE_Os_NEEDS_COMMENT (12, "-Wmaybe-uninitialized");
>        if (te != NULL
>  	  && csize == nb
>  	  && PTR_IS_ALIGNED (te, alignment))
>  	return tag_new_usable (tcache_get_n (tc_idx, tep, mangled));
> +      DIAG_POP_NEEDS_COMMENT;
>      }
>    return NULL;
>  }

Needs a 's/maya be/maybe/' to fix a typo in your comment. Otherwise,
looks good, and the comment makes it clear.

Reviewed-by: Collin Funk <collin.funk1@gmail.com>

Collin


More information about the Libc-alpha mailing list