[Bug testsuite/34168] New: dg-require-effective-target vect_double doesn't work

[Bug testsuite/34168] New: dg-require-effective-target vect_double doesn't work

[tkoenig at gcc dot gnu dot org]

I get

FAIL: gfortran.dg/vect/fast-math-pr33299.f90 execution test

Apparently, the check for  dg-require-effective-target vect_double
doesn't work.

This is on my Athlon XP 2600+, which does not have SSE2.  The
options for that test case do include the -msse2 flag, which is wrong:

Executing on host:
/home/ig25/gcc-bin/trunk/gcc/testsuite/gfortran/../../gfortran
-B/home/ig25/gcc-bin/trunk/gcc/tests
uite/gfortran/../../
/home/ig25/gcc/trunk/gcc/testsuite/gfortran.dg/vect/fast-math-pr33299.f90   -O2
-ftree-vectorize 
-ftree-vectorizer-verbose=4 -fdump-tree-vect-stats -msse2 -ffast-math 
-L/home/ig25/gcc-bin/trunk/i686-pc-linux-gnu/./
libgfortran/.libs
-L/home/ig25/gcc-bin/trunk/i686-pc-linux-gnu/./libgfortran/.libs
-L/home/ig25/gcc-bin/trunk/i686-pc-
linux-gnu/./libiberty  -lm   -o ./fast-math-pr33299.exe    (timeout = 300)

When I run the options by hand, I get an "illegal instruction"
error:

 gfortran -O2 -ftree-vectorize -ftree-vectorizer-verbose=4
-fdump-tree-vect-stats -msse2 -ffast-math fast-math-pr33299.f90 
$ ./a.out
Illegal instruction

$ cat /proc/cpuinfo 
processor       : 0
vendor_id       : AuthenticAMD
cpu family      : 6
model           : 8
model name      : AMD Athlon(TM) XP 2600+
stepping        : 1
cpu MHz         : 2083.200
cache size      : 256 KB
fdiv_bug        : no
hlt_bug         : no
f00f_bug        : no
coma_bug        : no
fpu             : yes
fpu_exception   : yes
cpuid level     : 1
wp              : yes
flags           : fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov
pat pse36 mmx fxsr sse syscall mmxext 3dnowext 3dnow up ts
bogomips        : 4170.03
clflush size    : 32


-- 
           Summary: dg-require-effective-target vect_double doesn't work
           Product: gcc
           Version: 4.3.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: testsuite
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: tkoenig at gcc dot gnu dot org


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

[Bug testsuite/34168] dg-require-effective-target vect_double doesn't work

[ubizjak at gmail dot com]

------- Comment #1 from ubizjak at gmail dot com  2007-11-21 07:18 -------
No, the problem is that vectorizer runtime tests require some special
treatment. You can look at gcc.dg/vect/vect-2.c, where check_vect is called to
check for SSE2 support (on x86 targets).

I think that kind of C glue function that would call check_vect() from
gcc.dg/vect/tree-vect.h is needed for fortran runtime vectorizer tests.


-- 

ubizjak at gmail dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2007-11-21 07:18:47
               date|                            |


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

[Bug testsuite/34168] runtime tests in gfortran.dg/vect fail for unsupported [non-SSE2] targets

[ubizjak at gmail dot com]

------- Comment #2 from ubizjak at gmail dot com  2007-11-21 17:40 -------
Changed summary to reflect real cause of failures.


-- 

ubizjak at gmail dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|dg-require-effective-target |runtime tests in
                   |vect_double doesn't work    |gfortran.dg/vect fail for
                   |                            |unsupported [non-SSE2]
                   |                            |targets


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

[Bug testsuite/34168] runtime tests in gfortran.dg/vect fail for unsupported [non-SSE2] targets

[victork at gcc dot gnu dot org]

------- Comment #3 from victork at gcc dot gnu dot org  2008-03-19 09:31 -------
*** Bug 34878 has been marked as a duplicate of this bug. ***


-- 

victork at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |kargl at gcc dot gnu dot org


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

[Bug testsuite/34168] runtime tests in gfortran.dg/vect fail for unsupported [non-SSE2] targets

[victork at gcc dot gnu dot org]

------- Comment #4 from victork at gcc dot gnu dot org  2008-03-19 10:37 -------
I've submitted a patch http://gcc.gnu.org/ml/gcc-patches/2008-03/msg01129.html
to fix this in testsuite.


-- 

victork at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |victork at gcc dot gnu dot
                   |dot org                     |org
             Status|NEW                         |ASSIGNED
   Last reconfirmed|2008-02-26 17:52:23         |2008-03-19 10:37:28
               date|                            |


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

[Bug testsuite/34168] runtime tests in gfortran.dg/vect fail for unsupported [non-SSE2] targets

[uros at gcc dot gnu dot org]

------- Comment #5 from uros at gcc dot gnu dot org  2008-03-20 19:44 -------
Subject: Bug 34168

Author: uros
Date: Thu Mar 20 19:43:56 2008
New Revision: 133393

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=133393
Log:
        PR testsuite/34168
        * lib/target-supports.exp (check_sse2_hw_available): New procedure.
        * gcc.dg/vect/vect.exp: Set dg-do-what-default to "compile"
        if SSE2 hardware is not available.
        * g++.dg/vect/vect.exp: Update target-dependent overrides to match
        gcc.dg/vect/vect.exp.
        * gfortran.dg/vect/vect.exp: Ditto.
        (check_effective_target_lp64_or_vect_no_align): Remove procedure.


Modified:
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/testsuite/g++.dg/vect/vect.exp
    trunk/gcc/testsuite/gcc.dg/vect/vect.exp
    trunk/gcc/testsuite/gfortran.dg/vect/vect.exp
    trunk/gcc/testsuite/lib/target-supports.exp


-- 


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

[Bug testsuite/34168] runtime tests in gfortran.dg/vect fail for unsupported [non-SSE2] targets

[uros at gcc dot gnu dot org]

------- Comment #6 from uros at gcc dot gnu dot org  2008-03-21 09:19 -------
Subject: Bug 34168

Author: uros
Date: Fri Mar 21 09:18:37 2008
New Revision: 133416

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=133416
Log:
        Backport from mainline:
        2008-03-20  Victor Kaplansky  <victork@gcc.gnu.org>
                    Uros Bizjak  <ubizjak@gmail.com>

        PR testsuite/34168
        * lib/target-supports.exp (check_sse2_hw_available): New procedure.
        * gcc.dg/vect/vect.exp: Set dg-do-what-default to "compile"
        if SSE2 hardware is not available.
        * g++.dg/vect/vect.exp: Update target-dependent overrides to match
        gcc.dg/vect/vect.exp.
        * gfortran.dg/vect/vect.exp: Ditto.
        (check_effective_target_lp64_or_vect_no_align): Remove procedure.

        2008-03-20  Uros Bizjak  <ubizjak@gmail.com>

        * gcc.dg/gomp/pr34692.c: Cleanup gimple dump file.


Modified:
    branches/gcc-4_3-branch/gcc/testsuite/ChangeLog
    branches/gcc-4_3-branch/gcc/testsuite/g++.dg/vect/vect.exp
    branches/gcc-4_3-branch/gcc/testsuite/gcc.dg/gomp/pr34692.c
    branches/gcc-4_3-branch/gcc/testsuite/gcc.dg/vect/vect.exp
    branches/gcc-4_3-branch/gcc/testsuite/gfortran.dg/vect/vect.exp
    branches/gcc-4_3-branch/gcc/testsuite/lib/target-supports.exp


-- 


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

[Bug testsuite/34168] runtime tests in gfortran.dg/vect fail for unsupported [non-SSE2] targets

[ubizjak at gmail dot com]

------- Comment #7 from ubizjak at gmail dot com  2008-03-21 09:25 -------
Fixed.


-- 

ubizjak at gmail dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|---                         |4.3.1


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