RFC: redundant stores in C++

Dale Johannesen dalej@apple.com
Tue Oct 4 18:15:00 GMT 2005


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
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: t2.diff.txt
URL: <https://gcc.gnu.org/pipermail/gcc/attachments/20051004/219d2c6c/attachment.txt>
-------------- next part --------------


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.)


More information about the Gcc mailing list