On Wed, Jul 11, 2012 at 12:06 PM, Richard Guenther <richard.guent...@gmail.com> wrote: > On Wed, Jul 11, 2012 at 11:41 AM, Steven Bosscher <stevenb....@gmail.com> > wrote: >> Hello, >> >> One for under the "eat your own dog food" header. >> >> Bootstrapped and tested on powerpc64-unknown-linux-gnu. >> cc1 is a few bytes smaller. I expect the benefits to be more >> noticeable with an LTO-enabled bootstrap, but I haven't figured out >> yet how to do that :-) >> >> OK for trunk? > > Hmm, we assume that the contents of the allocated memory is > undefined (zero content is ok practically). So if there is no hidden > init code anywhere in those functions the patch is ok.
Yes, I was careful to watch out for that. This is why e.g. ggc_alloc_string_stat isn't an ATTRIBUTE_MALLOC function. The only thing I wasn't sure about, is whether the poisoning of memory with ENABLE_GC_CHECKING counts as undefined. Ciao! Steven
