Re: [PATCH] Fix PR56521

2013-03-05 Thread Jakub Jelinek
On Tue, Mar 05, 2013 at 12:57:41PM +0100, Richard Biener wrote: > As follows. > > Richard. > > 2013-03-05 Richard Biener > > * tree-ssa-sccvn.c (set_value_id_for_result): For a NULL > result set a new value-id. Looks much better. You forgot to adjust the ChangeLog entry, and PR l

Re: [PATCH] Fix PR56521

2013-03-05 Thread Richard Biener
On Tue, 5 Mar 2013, Jakub Jelinek wrote: > On Tue, Mar 05, 2013 at 12:51:09PM +0100, Richard Biener wrote: > > VN now inserts all sorts of calls into the references hashtable, > > not only those which produce a value. This results in missing > > initializations of ->value_id which eventually PRE

Re: [PATCH] Fix PR56521

2013-03-05 Thread Jakub Jelinek
On Tue, Mar 05, 2013 at 12:51:09PM +0100, Richard Biener wrote: > VN now inserts all sorts of calls into the references hashtable, > not only those which produce a value. This results in missing > initializations of ->value_id which eventually PRE ends up > accessing. > > The following fixes that

[PATCH] Fix PR56521

2013-03-05 Thread Richard Biener
VN now inserts all sorts of calls into the references hashtable, not only those which produce a value. This results in missing initializations of ->value_id which eventually PRE ends up accessing. The following fixes that. Bootstrap and regtest pending on x86_64-unknown-linux-gnu. Richard. 20