* [patch, testsuite] Fix vectorizer testcase
@ 2007-07-01 11:54 Ira Rosen
0 siblings, 0 replies; 4+ messages in thread
From: Ira Rosen @ 2007-07-01 11:54 UTC (permalink / raw)
To: gcc-patches
In vect-strided-store-u16-i4.c there was a conditional call to abort() in
initialization loop in order to avoid its vectorization. However, now the
call is optimized out before the vectorizer and the loop gets vectorized
anyway. I removed the call to abort() and fixed dg-final checks.
Tested on ppc-linux. Committed as obvious.
Ira
ChangeLog:
* gcc.dg/vect/vect-strided-store-u16-i4.c: Remove abort() from input
initialization and fix dg-final check - the initialization loop is
vectorizable.
Index: testsuite/gcc.dg/vect/vect-strided-store-u16-i4.c
===================================================================
--- testsuite/gcc.dg/vect/vect-strided-store-u16-i4.c (revision 126162)
+++ testsuite/gcc.dg/vect/vect-strided-store-u16-i4.c (working copy)
@@ -58,8 +58,6 @@ int main (void)
a[i] = i;
b[i] = i * 2;
c[i] = 17;
- if (a[i] == 178)
- abort();
}
main1 (arr);
@@ -67,7 +65,8 @@ int main (void)
return 0;
}
-/* { dg-final { scan-tree-dump-times "vectorized 1 loops" 1 "vect" {
target { vect_interleave } } } } */
+/* { dg-final { scan-tree-dump-times "vectorized 1 loops" 2 "vect" {
target { vect_interleave } } } } */
+/* { dg-final { scan-tree-dump-times "vectorized 1 loops" 1 "vect" {
target { ! { vect_interleave } } } } } */
/* { dg-final { cleanup-tree-dump "vect" } } */
^ permalink raw reply [flat|nested] 4+ messages in thread
* [patch, testsuite] Fix vectorizer testcase
@ 2007-10-30 9:06 Revital1 Eres
0 siblings, 0 replies; 4+ messages in thread
From: Revital1 Eres @ 2007-10-30 9:06 UTC (permalink / raw)
To: Dorit Nuzman; +Cc: gcc-patches
OK for mainline?
Thanks,
Revital
* testsuite/gcc.dg/vect/pr33866.c: Require vect_int.
Index: testsuite/gcc.dg/vect/pr33866.c
===================================================================
--- testsuite/gcc.dg/vect/pr33866.c (revision 129765)
+++ testsuite/gcc.dg/vect/pr33866.c (working copy)
@@ -1,5 +1,6 @@
/* Testcase by Martin Michlmayr <tbm@cyrius.com> */
/* { dg-do compile } */
+/* { dg-require-effective-target vect_int } */
typedef struct
{
^ permalink raw reply [flat|nested] 4+ messages in thread
[parent not found: <OF300CE215.FBC49A92-ONC2257384.00275B12-C2257384.0027B467@LocalDomain>]
* Re: [patch, testsuite] Fix vectorizer testcase
[not found] <OF300CE215.FBC49A92-ONC2257384.00275B12-C2257384.0027B467@LocalDomain>
@ 2007-10-30 9:38 ` Dorit Nuzman
0 siblings, 0 replies; 4+ messages in thread
From: Dorit Nuzman @ 2007-10-30 9:38 UTC (permalink / raw)
To: Revital1 Eres; +Cc: gcc-patches
Revital1 Eres/Haifa/IBM wrote on 30/10/2007 00:13:40:
> OK for mainline?
>
it actually requires vect_long rather than vect_int.
I tested this fix on powerpc64-linux with -m32 and -m64.
A fix to require vect_long is pre-approved.
dorit
> Thanks,
> Revital
>
> * testsuite/gcc.dg/vect/pr33866.c: Require vect_int.
>
> Index: testsuite/gcc.dg/vect/pr33866.c
> ===================================================================
> --- testsuite/gcc.dg/vect/pr33866.c (revision 129765)
> +++ testsuite/gcc.dg/vect/pr33866.c (working copy)
> @@ -1,5 +1,6 @@
> /* Testcase by Martin Michlmayr <tbm@cyrius.com> */
> /* { dg-do compile } */
> +/* { dg-require-effective-target vect_int } */
>
> typedef struct
> {
^ permalink raw reply [flat|nested] 4+ messages in thread
[parent not found: <OF0E1152B9.78AD8AFA-ON88257384.002C2385-88257384.002D41FF@LocalDomain>]
* Re: [patch, testsuite] Fix vectorizer testcase
[not found] <OF0E1152B9.78AD8AFA-ON88257384.002C2385-88257384.002D41FF@LocalDomain>
@ 2007-10-30 10:44 ` Revital1 Eres
0 siblings, 0 replies; 4+ messages in thread
From: Revital1 Eres @ 2007-10-30 10:44 UTC (permalink / raw)
To: Dorit Nuzman; +Cc: gcc-patches
Committed to r129767.
Thanks,
Revital
Dorit Nuzman/Haifa/IBM wrote on 30/10/2007 10:14:20:
> Revital1 Eres/Haifa/IBM wrote on 30/10/2007 00:13:40:
>
> > OK for mainline?
> >
>
> it actually requires vect_long rather than vect_int.
> I tested this fix on powerpc64-linux with -m32 and -m64.
> A fix to require vect_long is pre-approved.
>
> dorit
>
> > Thanks,
> > Revital
> >
> > * testsuite/gcc.dg/vect/pr33866.c: Require vect_int.
> >
> > Index: testsuite/gcc.dg/vect/pr33866.c
> > ===================================================================
> > --- testsuite/gcc.dg/vect/pr33866.c (revision 129765)
> > +++ testsuite/gcc.dg/vect/pr33866.c (working copy)
> > @@ -1,5 +1,6 @@
> > /* Testcase by Martin Michlmayr <tbm@cyrius.com> */
> > /* { dg-do compile } */
> > +/* { dg-require-effective-target vect_int } */
> >
> > typedef struct
> > {
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2007-10-30 8:27 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-07-01 11:54 [patch, testsuite] Fix vectorizer testcase Ira Rosen
2007-10-30 9:06 Revital1 Eres
[not found] <OF300CE215.FBC49A92-ONC2257384.00275B12-C2257384.0027B467@LocalDomain>
2007-10-30 9:38 ` Dorit Nuzman
[not found] <OF0E1152B9.78AD8AFA-ON88257384.002C2385-88257384.002D41FF@LocalDomain>
2007-10-30 10:44 ` Revital1 Eres
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).