[C/C++ PATCH] Don't emit invalid VEC_COND_EXPR for vector comparisons (PR c/68062)
Marek Polacek
polacek@redhat.com
Wed Mar 2 12:17:00 GMT 2016
On Wed, Mar 02, 2016 at 01:10:51PM +0100, Jakub Jelinek wrote:
> On Wed, Mar 02, 2016 at 12:43:07PM +0100, Richard Biener wrote:
> > On Thu, Jan 28, 2016 at 8:33 PM, Marek Polacek <polacek@redhat.com> wrote:
> > > On Thu, Jan 28, 2016 at 04:47:04AM -0800, H.J. Lu wrote:
> > >> I got
> > >>
> > >> FAIL: c-c++-common/vector-compare-4.c -Wc++-compat (test for
> > >> warnings, line 17)
> > >> FAIL: c-c++-common/vector-compare-4.c -Wc++-compat (test for
> > >> warnings, line 18)
> > >> FAIL: c-c++-common/vector-compare-4.c -Wc++-compat (test for
> > >> warnings, line 34)
> > >> FAIL: c-c++-common/vector-compare-4.c -Wc++-compat (test for
> > >> warnings, line 35)
> > >
> > > Ah, patch(1) bogosity, fixed now. Sorry about that.
> >
> > Note on i?86 I see (or x86_64 with -m32 -march=i586)
> >
> > /space/rguenther/src/svn/gcc-5-branch/gcc/testsuite/c-c++-common/vector-compare-4.c:29:1:
> > warning: SSE vector return without SSE enabled changes the ABI
> > [-Wpsabi]^M
> >
> > and thus the testcase fails (it has been adjusted for a similar kind
> > of error on ppc
> > on trunk but not yet on the branch(es)).
>
> Thus should fix that (your new PR70022 test fails similarly, but here we can
> use -w, on vector-compare-4.c which expects other warnings we need the prune
> for powerpc; though, as has been said earlier, it would be better if
> rs6000 started honoring -Wno-psabi and emitted inform rather than warning).
> Ok for trunk?
>
> 2016-03-02 Jakub Jelinek <jakub@redhat.com>
>
> PR c/68062
> * c-c++-common/vector-compare-4.c: Add -Wno-psabi to dg-options.
>
> PR middle-end/70022
> * gcc.dg/pr70022.c: Add -w -Wno-psabi to dg-options.
>
> --- gcc/testsuite/c-c++-common/vector-compare-4.c.jj 2016-02-03 23:36:38.000000000 +0100
> +++ gcc/testsuite/c-c++-common/vector-compare-4.c 2016-03-02 13:07:29.229073904 +0100
> @@ -1,6 +1,6 @@
> /* PR c/68062 */
> /* { dg-do compile } */
> -/* { dg-options "-Wsign-compare" } */
> +/* { dg-options "-Wsign-compare -Wno-psabi" } */
> /* Ignore warning on some powerpc configurations. */
> /* { dg-prune-output "non-standard ABI extension" } */
Can't approve, but I'm certainly fine with it. Thanks for fixing.
Marek
More information about the Gcc-patches
mailing list