Re: [patch] convert.c: Fix PR tree-optimization/25125. (Take 2)

[Dorit Naishlos <DORIT at il dot ibm dot com>]

Now that Kazu's patch went in, can this patch -
http://gcc.gnu.org/ml/gcc-patches/2005-12/msg01618.html
be approved for mainline?

thanks,
dorit

        * gcc.dg/vect/vect.exp: Add -fwrapv flag when compiling testcases
        prefixed with "wrapv-".
        * gcc.dg/vect/wrapv-vect-7.c: New (wrapv version of vect-7.c).
        * gcc.dg/vect/wrapv-vect-35.c: New (wrapv version of vect-35.c).
        * gcc.dg/vect/wrapv-vect-reduc-2char.c: New (wrapv version of
        vect-reduc-2char.c).
        * gcc.dg/vect/wrapv-vect-reduc-2short.c: New (wrapv version of
        vect-reduc-2short.c).

:ADDPATCH testsuite (vectorizer):

(See attached file: wrapv.patch.txt)



> Kazu Hirata <kazu@codesourcery.com> wrote on 21/12/2005 19:29:49:
>
> > Thank you for a coming up with a patch.  Are you going to ask for an
> > approval of
> > your patch?  IMHO, it is a good patch with or without my patch.
> >
>
> without your patch the new testcases duplicate the behavior of the
current
> testcases, so I think I'll submit this patch for approval after your
patch
> goes in
>
> dorit
>
> > I guess we can open a missed optimization PR as you said.  Technically,
> we
> > cannot narrow the type of a signed expression, but (hopefully)
> > nobody is going
> > to mess with vectorized expressions in a way that depends on whether
> > -fwrapv is
> > given or not.
> >
> > Kazu Hirata
>

Attachment: wrapv.patch.txt
Description: Text document