[Bug testsuite/47057] New: FAIL/XPASS gcc.dg/vect/costmodel/ppc/costmodel-vect-outer-fir.c

[Bug testsuite/47057] New: FAIL/XPASS gcc.dg/vect/costmodel/ppc/costmodel-vect-outer-fir.c

[dominiq at lps dot ens.fr]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47057

           Summary: FAIL/XPASS
                    gcc.dg/vect/costmodel/ppc/costmodel-vect-outer-fir.c
           Product: gcc
           Version: 4.6.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: testsuite
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: dominiq@lps.ens.fr
            Target: powerpc*-*-*


The test gcc.dg/vect/costmodel/ppc/costmodel-vect-outer-fir.c gives

XPASS: gcc.dg/vect/costmodel/ppc/costmodel-vect-outer-fir.c
scan-tree-dump-times vect "OUTER LOOP VECTORIZED" 2
FAIL: gcc.dg/vect/costmodel/ppc/costmodel-vect-outer-fir.c scan-tree-dump-times
vect "OUTER LOOP VECTORIZED" 1

Obviously if there are two outer loops vectorized, there cannot be only one
vectorized! The following patch fixes the test

---
../_gcc_clean/gcc/testsuite/gcc.dg/vect/costmodel/ppc/costmodel-vect-outer-fir.c
   2007-11-21 20:18:34.000000000 +0100
+++
../gcc-4.6-work/gcc/testsuite/gcc.dg/vect/costmodel/ppc/costmodel-vect-outer-fir.c
   2010-12-23 22:02:28.000000000 +0100
@@ -71,6 +71,6 @@ int main (void)
   return 0;
 }

-/* { dg-final { scan-tree-dump-times "OUTER LOOP VECTORIZED" 2 "vect" { xfail
*-*-* } } } */
-/* { dg-final { scan-tree-dump-times "OUTER LOOP VECTORIZED" 1 "vect" { xfail
vect_no_align } } } */
+/* { dg-final { scan-tree-dump-times "OUTER LOOP VECTORIZED" 2 "vect" { xfail
vect_no_align } } } */
+/* { dg-final { scan-tree-dump-times "OUTER LOOP VECTORIZED" 1 "vect" { xfail
{ ! vect_no_align } } } } */
 /* { dg-final { cleanup-tree-dump "vect" } } */

[Bug testsuite/47057] FAIL/XPASS gcc.dg/vect/costmodel/ppc/costmodel-vect-outer-fir.c

[irar at il dot ibm.com]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47057

Ira Rosen <irar at il dot ibm.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |ASSIGNED
   Last reconfirmed|                            |2010.12.25 11:49:58
                 CC|                            |irar at il dot ibm.com
         AssignedTo|unassigned at gcc dot       |irar at gcc dot gnu.org
                   |gnu.org                     |
     Ever Confirmed|0                           |1

--- Comment #1 from Ira Rosen <irar at il dot ibm.com> 2010-12-25 11:49:58 UTC ---
The first loop now gets vectorized, it didn't because of:

/* Currently not vectorized because we get too many BBs in the inner-loop,
   because the compiler doesn't realize that the inner-loop executes at
   least once (cause k<4), and so there's no need to create a guard code
   to skip the inner-loop in case it doesn't execute.  */


So, the correct fix is to expect 2 outer loop vectorized on targets that
support realignment.

I'll fix the test.

Ira

[Bug testsuite/47057] FAIL/XPASS gcc.dg/vect/costmodel/ppc/costmodel-vect-outer-fir.c

[irar at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47057

--- Comment #2 from irar at gcc dot gnu.org 2010-12-25 12:54:02 UTC ---
Author: irar
Date: Sat Dec 25 12:53:58 2010
New Revision: 168242

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=168242
Log:

        PR testsuite/47057
        * gcc.dg/vect/costmodel/ppc/costmodel-vect-outer-fir.c: Change
        comment. Expect two outer loops to get vectorized.


Modified:
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/testsuite/gcc.dg/vect/costmodel/ppc/costmodel-vect-outer-fir.c

[Bug testsuite/47057] FAIL/XPASS gcc.dg/vect/costmodel/ppc/costmodel-vect-outer-fir.c

[irar at il dot ibm.com]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47057

Ira Rosen <irar at il dot ibm.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED

--- Comment #3 from Ira Rosen <irar at il dot ibm.com> 2010-12-25 12:55:18 UTC ---
Fixed.