Re: RFC: redundant stores in C++

[Dale Johannesen <dalej at apple dot com>]

On Oct 1, 2005, at 8:41 PM, Andrew Pinski wrote:
> On Oct 1, 2005, at 11:10 PM, Dale Johannesen wrote:
> > > But better fix would be not call split_nonconstant_init_1 for
> > > local decls and have the front-end produce a CONSTRUCTOR which is
> > > just like what the C front-end produces.
> >
> > I'll try it.
>
> This patch should fix the problem and also fixes FSF PR 8045 at
> the same time.  FSF PR 8045 is about an missing unused variable
> causes by this code.
>
> This patch causes us to be more similar with the C front-end.  It
> should also cause us save us some compile time issue when gimplifing
> and memory too.
>
> Note I have not tested this yet by either looking at the code gen
> or even compiling it.
>
> I will be doing a bootstrap/test of this right now.
>
> -- Pinski

Attachment: t2.diff.txt
Description: Text document

Thanks.  The last line of this patch should use "decl", not "dest".  
With that obvious
change it tests OK in apple's branch.  (I cannot build mainline on 
darwin
x86 at the moment for unrelated reasons - nothing that means there is
a problem in mainline, I don't think.)