This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH PR70729] The second part of patch.
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Thomas Schwinge <thomas at codesourcery dot com>
- Cc: Yuri Rumyantsev <ysrumyan at gmail dot com>, gcc-patches <gcc-patches at gcc dot gnu dot org>, Richard Biener <richard dot guenther at gmail dot com>, Igor Zamyatin <izamyatin at gmail dot com>, ÐÐÑÑ ÐÐÐÐÐÐÑ <enkovich dot gnu at gmail dot com>
- Date: Thu, 30 Jun 2016 16:28:44 +0200
- Subject: Re: [PATCH PR70729] The second part of patch.
- Authentication-results: sourceware.org; auth=none
- References: <CAEoMCqSx5+c2N9UpvbntYPaTxcBwzk3pAY+Kfc9wFVQp8qDMbA at mail dot gmail dot com> <87eg7gt1bw dot fsf at kepler dot schwinge dot homeip dot net> <CAEoMCqQ6WEO2p0DzbtSuxjbn10=b-0kJXx4tqzXuz16rb7=igg at mail dot gmail dot com> <20160630122145 dot GM7387 at tucnak dot redhat dot com> <CAEoMCqTqHsxOJ9J7NCH2=6qx-JiDpqPd3shoRKhw5Wn8U0QNgA at mail dot gmail dot com> <87eg7eoisz dot fsf at hertz dot schwinge dot homeip dot net>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Thu, Jun 30, 2016 at 04:25:16PM +0200, Thomas Schwinge wrote:
> Hi!
>
> On Thu, 30 Jun 2016 16:48:25 +0300, Yuri Rumyantsev <ysrumyan@gmail.com> wrote:
> > Thanks for your help.
> > Could you please look at the following simple patch which cures
> > regression - we need to nullify loop safelen field in
> > adjust_simduid_builtins:
> >
> > diff --git a/gcc/tree-vectorizer.c b/gcc/tree-vectorizer.c
> > index 2669813..f70380c 100644
> > --- a/gcc/tree-vectorizer.c
> > +++ b/gcc/tree-vectorizer.c
> > @@ -204,6 +204,10 @@ adjust_simduid_builtins (hash_table<simduid_to_vf> *htab)
> > gcc_assert (TREE_CODE (arg) == SSA_NAME);
> > simduid_to_vf *p = NULL, data;
> > data.simduid = DECL_UID (SSA_NAME_VAR (arg));
> > + /* Need to nullify safelen fielf of loop since it's vale is not
> > + valid after transformation. */
s/fielf/field/
s/vale/value/
> > + if (bb->loop_father && bb->loop_father->safelen > 0)
> > + bb->loop_father->safelen = 0;
> > if (htab)
> > {
> > p = htab->find (&data);
Jakub