This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] PR 38156
- From: Razya Ladelsky <RAZYA at il dot ibm dot com>
- To: Zdenek Dvorak <rakdver at kam dot mff dot cuni dot cz>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Wed, 3 Jun 2009 16:19:32 +0300
- Subject: Re: [PATCH] PR 38156
Zdenek Dvorak <rakdver@kam.mff.cuni.cz> wrote on 03/06/2009 15:58:22:
> Hi,
>
> > The patch committed to GCC4.4 following PR38156 was slightly different
> > than what was submitted here:
> > http://gcc.gnu.org/ml/gcc-patches/2008-11/msg00882.html, although it
> > fixed the failure as well.
> > (but creating unnecessary structures)
> >
> > I would like to commit the more appropriate patch now to GCC4.5, so
that
> > arg_struct and new_arg_struct will not be
> > built when there's no reason for it to happen.
>
> the patch is OK.
Thanks Zdenek,
and regarding the backporting?
In GCC4.3 ,the testcase is not failing by default, only when
-ftree-parallelize-loops
is enabled.
Thanks,
Razya
>
> > Index: tree-parloops.c
> > ===================================================================
> > --- tree-parloops.c (revision 147835)
> > +++ tree-parloops.c (working copy)
> > @@ -1193,7 +1193,7 @@
> >
> > VEC_free (basic_block, heap, body);
> >
> > - if (htab_elements (name_copies) == 0 && reduction_list == 0)
> > + if (htab_elements (name_copies) == 0 && htab_elements
> (reduction_list)==0)
> > {
>
> Add space before and after ==
>
> Zdenek