This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, take 2] Fix PR tree-optimization/49960 ,Fix self data dependence
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Razya Ladelsky <RAZYA at il dot ibm dot com>
- Cc: Richard Guenther <richard dot guenther at gmail dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Mon, 21 Nov 2011 13:57:07 +0100
- Subject: Re: [PATCH, take 2] Fix PR tree-optimization/49960 ,Fix self data dependence
- References: <OF662F6234.DE8C60EA-ONC2257926.0048D53A-C225792C.00231C15@il.ibm.com> <CAFiYyc15E0jqzPC+SD40cWTj18Tes0ZbEm56BXHHe-eZnqSRvw@mail.gmail.com> <OF746BCB18.CF82809F-ONC225792E.0051CE3F-C225792E.00564975@il.ibm.com> <CAFiYyc2ykFPCW8A8vW=f5UbNa7zFRQObwL13D9ioXjCd_em9pQ@mail.gmail.com> <CAFiYyc0D8n6BU-CLqB132FBuxOqJdmiRBDTiZ6Spf8P0J5rbJg@mail.gmail.com> <OFE59B30C0.27072E7A-ONC2257949.0035FEEA-C2257949.0039D985@il.ibm.com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
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.
So, when compute_all_dependences no longer calls it, neither should
vect_analyze_data_refs.
Jakub