[U-Boot] [PATCH] dlmalloc: Ignore gcc4.4 compiler warnings

Wolfgang Denk wd at denx.de
Wed Jul 29 07:47:28 CEST 2009


Dear Scott,

In message <20090728225244.GA8187 at b07421-ec1.am.freescale.net> you wrote:
>
> The patch title is bad -- it's not disabling warnings, it's disabling an
> aspect of C99 that the code is incompatible with (and which is pretty
> questionable in the first place with such low level code). Note that in
> Linux, this is disabled for the entire kernel.

I know. But Linux is bigger than U-Boot, and I think we should be able
to fix the few isolated places that throw such warnings.

> As things stand, GCC may do bad things with that code.  With this patch,
> it may not (at least not this particular sort of bad thing).  Those bad
> things are not limited to the places where it warns -- those are just the
> violations it detected.

Agreed, and that's why I want to see this fixed.

> Do you have an alternative malloc implementation in mind that is designed
> to work with strict aliasing, or a suggested fix to the current one?

I did not look into this yet - there was some discussion about a
malloc replacement, but it faded away without visible result.

I cannot do everything myself, but I can oppose changes that are IMO
to the worse.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
I express preference for a chronological  sequence  of  events  which
precludes a violence.   - Terry Pratchett, _The Dark Side of the Sun_


More information about the U-Boot mailing list