gcc-patches-ow...@gcc.gnu.org wrote on 21/11/2011 02:57:07 PM: > From: Jakub Jelinek <ja...@redhat.com> > To: Razya Ladelsky/Haifa/IBM@IBMIL > Cc: Richard Guenther <richard.guent...@gmail.com>, GCC Patches <gcc- > patc...@gcc.gnu.org> > Date: 21/11/2011 02:57 PM > Subject: Re: [PATCH, take 2] Fix PR tree-optimization/49960 ,Fix > self data dependence > Sent by: gcc-patches-ow...@gcc.gnu.org > > On Tue, Nov 15, 2011 at 12:31:39PM +0200, Razya Ladelsky wrote: > > This patch fixes the failures described in > > http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49960 > > It also fixes bzips when run with autopar enabled. > > Sorry, I've been away and couldn't react to this earlier, but the patch > looks wrong to me. The reason why I've been calling compute_self_dependence > from the gather handling code is to do there what we did for normal data > references in compute_all_dependences, except for the affine stuff.
Hi Jakub, what do you mean by 'except for the affine stuff'? Before having this patch, compute_self_depepndence just marked the distance zero and returned, while now it calls compute_affine_dependence. So I think you do get the right outcome now. > So, when compute_all_dependences no longer calls it, neither should > vect_analyze_data_refs. So do you want to replace it with the call to compute_affine_dependence? Thanks, Razya > > Jakub >
