[PATCH, take 2] Fix PR tree-optimization/49960 ,Fix self data dependence
Razya Ladelsky
RAZYA@il.ibm.com
Mon Nov 21 14:24:00 GMT 2011
gcc-patches-owner@gcc.gnu.org wrote on 21/11/2011 02:57:07 PM:
> From: Jakub Jelinek <jakub@redhat.com>
> To: Razya Ladelsky/Haifa/IBM@IBMIL
> Cc: Richard Guenther <richard.guenther@gmail.com>, GCC Patches <gcc-
> patches@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-owner@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
>
More information about the Gcc-patches
mailing list