[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