This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Fix PR23477


Richard Guenther <rguenther@suse.de> writes:

> by special casing all-zero constructor.
> 
> Bootstrapped and tested on x86_64-unknown-linux-gnu.
> 
> Ok for mainline?
> 
> Thanks,
> Richard.
> 
> :ADDPATCH middle-end:
> 
> 
> 2005-08-30  Richard Guenther  <rguenther@suse.de>
> 
> 	PR middle-end/23477
> 	* expr.c (all_zeros_p): New function.
> 	(expand_expr_real_1): Handle the case of an all-zero
> 	non-addressable constructor separate.

This special case makes sense to me.  This is OK with the change to
use clear_storage, assuming that passes bootstrap and testing.

In the ChangeLog entry, it should say "separately" instead of
"seperate".  Or you could say "specially".

Thanks.

:REVIEWMAIL:

Ian


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]