public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug testsuite/44159]  New: CPU options cause testsuite failures
@ 2010-05-16 14:21 dougsemler at gmail dot com
  2010-05-16 14:23 ` [Bug testsuite/44159] " dougsemler at gmail dot com
                   ` (4 more replies)
  0 siblings, 5 replies; 6+ messages in thread
From: dougsemler at gmail dot com @ 2010-05-16 14:21 UTC (permalink / raw)
  To: gcc-bugs

On x86_64-w64-mingw32 targets, during tests such as avx-vaddpd-1.c, the
testsuite fails.  This seems to be due to the fact that the test function is
inlined into main.  Due to this inlining, some of the the xmm registers are
saved and restored.  However, because the -mavx flag is passed, the register
saves are performed using instructions not available on the CPU.  In other
words, the register save/restore is performed before the __cpuid function is
called to determine whether the test is supported on the machine that is
running the test.

This is hit-and-miss and is problematic on any tests that require unsupported
registry saves on this target type.


-- 
           Summary: CPU options cause testsuite failures
           Product: gcc
           Version: 4.5.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: testsuite
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: dougsemler at gmail dot com
GCC target triplet: x86_64-w64-mingw32


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


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug testsuite/44159] CPU options cause testsuite failures
  2010-05-16 14:21 [Bug testsuite/44159] New: CPU options cause testsuite failures dougsemler at gmail dot com
@ 2010-05-16 14:23 ` dougsemler at gmail dot com
  2010-05-17 14:55 ` ktietz at gcc dot gnu dot org
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: dougsemler at gmail dot com @ 2010-05-16 14:23 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #1 from dougsemler at gmail dot com  2010-05-16 14:23 -------
Created an attachment (id=20672)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20672&action=view)
Assembly that shows unsupported registry saves

This assembly is made from:
 /opt/build/x86_64-pc-linux/x86_64-w64-mingw32/gcc/gcc/xgcc
-B/opt/build/x86_64-pc-linux/x86_64-w64-mingw32/gcc/gcc/
/opt/src/repos/gcc.git/gcc/testsuite/gcc.target/i386/avx-vaddpd-1.c  -O2 -mavx 
-lm  -o ./avx-vaddpd-1.exe

during a test suite run.  As can be seen, the vmovdqa instruction is used to
save the registers xmm6,7, and 8 prior to the __cpuid test...which is not
available on my core2 duo.


-- 


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


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug testsuite/44159] CPU options cause testsuite failures
  2010-05-16 14:21 [Bug testsuite/44159] New: CPU options cause testsuite failures dougsemler at gmail dot com
  2010-05-16 14:23 ` [Bug testsuite/44159] " dougsemler at gmail dot com
@ 2010-05-17 14:55 ` ktietz at gcc dot gnu dot org
  2010-06-07 10:57 ` ktietz at gcc dot gnu dot org
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: ktietz at gcc dot gnu dot org @ 2010-05-17 14:55 UTC (permalink / raw)
  To: gcc-bugs



-- 

ktietz at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2010-05-17 14:54:58
               date|                            |


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


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug testsuite/44159] CPU options cause testsuite failures
  2010-05-16 14:21 [Bug testsuite/44159] New: CPU options cause testsuite failures dougsemler at gmail dot com
  2010-05-16 14:23 ` [Bug testsuite/44159] " dougsemler at gmail dot com
  2010-05-17 14:55 ` ktietz at gcc dot gnu dot org
@ 2010-06-07 10:57 ` ktietz at gcc dot gnu dot org
  2010-06-07 11:09 ` ktietz at gcc dot gnu dot org
  2010-06-07 11:09 ` ktietz at gcc dot gnu dot org
  4 siblings, 0 replies; 6+ messages in thread
From: ktietz at gcc dot gnu dot org @ 2010-06-07 10:57 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #2 from ktietz at gcc dot gnu dot org  2010-06-07 10:57 -------
Subject: Bug 44159

Author: ktietz
Date: Mon Jun  7 10:56:44 2010
New Revision: 160363

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=160363
Log:
2010-06-07  Kai Tietz  <kai.tietz@onevision.com>

        PR target/44159
        * gcc.target/i386/abi-2.c: Check sysv abi here.
        * gcc.target/i386/aes-avx-check.h: Call test in noinline
        function to avoid failures by different ABIs.
        * gcc.target/i386/aes-check.h: Likewise.
        * gcc.target/i386/avx-check.h: Likewise.
        * gcc.target/i386/fma4-check.h: Likewise.
        * gcc.target/i386/mmx-3dnow-check.h: Likewise.
        * gcc.target/i386/mmx-check.h: Likewise.
        * gcc.target/i386/pclmul-avx-check.h: Likewise.
        * gcc.target/i386/pclmul-check.h: Likewise.
        * gcc.target/i386/sse-check.h: Likewise.
        * gcc.target/i386/sse2-check.h: Likewise.
        * gcc.target/i386/sse3-check.h: Likewise.
        * gcc.target/i386/sse4_1-check.h: Likewise.
        * gcc.target/i386/sse4_2-check.h: Likewise.
        * gcc.target/i386/sse4a-check.h: Likewise.
        * gcc.target/i386/ssse3-check.h: Likewise.
        * gcc.target/i386/xop-check.h: Likewise.
        * gcc.target/i386/pr27971.c: Fix for LLP64.
        * gcc.target/i386/pr39139.c: Likewise.
        * gcc.target/i386/pr39315-check.c: Likewise.
        * gcc.target/i386/vararg-1.c: Likewise.
        * gcc.target/i386/vararg-2.c: Likewise.
        Additional add dg-compile to avoid failure due
        missing foo symbol.


Modified:
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/testsuite/gcc.target/i386/abi-2.c
    trunk/gcc/testsuite/gcc.target/i386/aes-avx-check.h
    trunk/gcc/testsuite/gcc.target/i386/aes-check.h
    trunk/gcc/testsuite/gcc.target/i386/avx-check.h
    trunk/gcc/testsuite/gcc.target/i386/fma4-check.h
    trunk/gcc/testsuite/gcc.target/i386/mmx-3dnow-check.h
    trunk/gcc/testsuite/gcc.target/i386/mmx-check.h
    trunk/gcc/testsuite/gcc.target/i386/pclmul-avx-check.h
    trunk/gcc/testsuite/gcc.target/i386/pclmul-check.h
    trunk/gcc/testsuite/gcc.target/i386/pr27971.c
    trunk/gcc/testsuite/gcc.target/i386/pr39139.c
    trunk/gcc/testsuite/gcc.target/i386/pr39315-check.c
    trunk/gcc/testsuite/gcc.target/i386/sse-check.h
    trunk/gcc/testsuite/gcc.target/i386/sse2-check.h
    trunk/gcc/testsuite/gcc.target/i386/sse3-check.h
    trunk/gcc/testsuite/gcc.target/i386/sse4_1-check.h
    trunk/gcc/testsuite/gcc.target/i386/sse4_2-check.h
    trunk/gcc/testsuite/gcc.target/i386/sse4a-check.h
    trunk/gcc/testsuite/gcc.target/i386/ssse3-check.h
    trunk/gcc/testsuite/gcc.target/i386/vararg-1.c
    trunk/gcc/testsuite/gcc.target/i386/vararg-2.c
    trunk/gcc/testsuite/gcc.target/i386/xop-check.h


-- 


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


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug testsuite/44159] CPU options cause testsuite failures
  2010-05-16 14:21 [Bug testsuite/44159] New: CPU options cause testsuite failures dougsemler at gmail dot com
                   ` (2 preceding siblings ...)
  2010-06-07 10:57 ` ktietz at gcc dot gnu dot org
@ 2010-06-07 11:09 ` ktietz at gcc dot gnu dot org
  2010-06-07 11:09 ` ktietz at gcc dot gnu dot org
  4 siblings, 0 replies; 6+ messages in thread
From: ktietz at gcc dot gnu dot org @ 2010-06-07 11:09 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #3 from ktietz at gcc dot gnu dot org  2010-06-07 11:09 -------
Subject: Bug 44159

Author: ktietz
Date: Mon Jun  7 11:08:46 2010
New Revision: 160365

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=160365
Log:
2010-06-07  Kai Tietz  <kai.tietz@onevision.com>

        Backport from mainline:
        PR target/44159
        * gcc.target/i386/abi-2.c: Check sysv abi here.
        * gcc.target/i386/aes-avx-check.h: Call test in noinline
        function to avoid failures by different ABIs.
        * gcc.target/i386/aes-check.h: Likewise.
        * gcc.target/i386/avx-check.h: Likewise.
        * gcc.target/i386/fma4-check.h: Likewise.
        * gcc.target/i386/mmx-3dnow-check.h: Likewise.
        * gcc.target/i386/mmx-check.h: Likewise.
        * gcc.target/i386/pclmul-avx-check.h: Likewise.
        * gcc.target/i386/pclmul-check.h: Likewise.
        * gcc.target/i386/sse-check.h: Likewise.
        * gcc.target/i386/sse2-check.h: Likewise.
        * gcc.target/i386/sse3-check.h: Likewise.
        * gcc.target/i386/sse4_1-check.h: Likewise.
        * gcc.target/i386/sse4_2-check.h: Likewise.
        * gcc.target/i386/sse4a-check.h: Likewise.
        * gcc.target/i386/ssse3-check.h: Likewise.
        * gcc.target/i386/xop-check.h: Likewise.
        * gcc.target/i386/pr27971.c: Fix for LLP64.
        * gcc.target/i386/pr39139.c: Likewise.
        * gcc.target/i386/pr39315-check.c: Likewise.
        * gcc.target/i386/vararg-1.c: Likewise.
        * gcc.target/i386/vararg-2.c: Likewise.
        Additional add dg-compile to avoid failure due
        missing foo symbol.

        * gcc.dg/compound-literal-1.c: Fix for llp64.
        * gcc.dg/pr32370.c: Likewise.
        * gcc.dg/pr37561.c: Likewise.
        * gcc.dg/pr41340.c: Likewise.
        * gcc.dg/pr41551.c: Likewise.


Modified:
    branches/gcc-4_5-branch/gcc/testsuite/ChangeLog
    branches/gcc-4_5-branch/gcc/testsuite/gcc.dg/compound-literal-1.c
    branches/gcc-4_5-branch/gcc/testsuite/gcc.dg/pr32370.c
    branches/gcc-4_5-branch/gcc/testsuite/gcc.dg/pr37561.c
    branches/gcc-4_5-branch/gcc/testsuite/gcc.dg/pr41340.c
    branches/gcc-4_5-branch/gcc/testsuite/gcc.dg/pr41551.c
    branches/gcc-4_5-branch/gcc/testsuite/gcc.target/i386/abi-2.c
    branches/gcc-4_5-branch/gcc/testsuite/gcc.target/i386/aes-avx-check.h
    branches/gcc-4_5-branch/gcc/testsuite/gcc.target/i386/aes-check.h
    branches/gcc-4_5-branch/gcc/testsuite/gcc.target/i386/avx-check.h
    branches/gcc-4_5-branch/gcc/testsuite/gcc.target/i386/fma4-check.h
    branches/gcc-4_5-branch/gcc/testsuite/gcc.target/i386/mmx-3dnow-check.h
    branches/gcc-4_5-branch/gcc/testsuite/gcc.target/i386/mmx-check.h
    branches/gcc-4_5-branch/gcc/testsuite/gcc.target/i386/pclmul-avx-check.h
    branches/gcc-4_5-branch/gcc/testsuite/gcc.target/i386/pclmul-check.h
    branches/gcc-4_5-branch/gcc/testsuite/gcc.target/i386/pr27971.c
    branches/gcc-4_5-branch/gcc/testsuite/gcc.target/i386/pr39139.c
    branches/gcc-4_5-branch/gcc/testsuite/gcc.target/i386/pr39315-check.c
    branches/gcc-4_5-branch/gcc/testsuite/gcc.target/i386/sse-check.h
    branches/gcc-4_5-branch/gcc/testsuite/gcc.target/i386/sse2-check.h
    branches/gcc-4_5-branch/gcc/testsuite/gcc.target/i386/sse3-check.h
    branches/gcc-4_5-branch/gcc/testsuite/gcc.target/i386/sse4_1-check.h
    branches/gcc-4_5-branch/gcc/testsuite/gcc.target/i386/sse4_2-check.h
    branches/gcc-4_5-branch/gcc/testsuite/gcc.target/i386/sse4a-check.h
    branches/gcc-4_5-branch/gcc/testsuite/gcc.target/i386/ssse3-check.h
    branches/gcc-4_5-branch/gcc/testsuite/gcc.target/i386/vararg-1.c
    branches/gcc-4_5-branch/gcc/testsuite/gcc.target/i386/vararg-2.c
    branches/gcc-4_5-branch/gcc/testsuite/gcc.target/i386/xop-check.h


-- 


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


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug testsuite/44159] CPU options cause testsuite failures
  2010-05-16 14:21 [Bug testsuite/44159] New: CPU options cause testsuite failures dougsemler at gmail dot com
                   ` (3 preceding siblings ...)
  2010-06-07 11:09 ` ktietz at gcc dot gnu dot org
@ 2010-06-07 11:09 ` ktietz at gcc dot gnu dot org
  4 siblings, 0 replies; 6+ messages in thread
From: ktietz at gcc dot gnu dot org @ 2010-06-07 11:09 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #4 from ktietz at gcc dot gnu dot org  2010-06-07 11:09 -------
Fixed an 4.5 branch and on mainline.


-- 

ktietz at gcc dot gnu dot org changed:

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


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


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2010-06-07 11:09 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-05-16 14:21 [Bug testsuite/44159] New: CPU options cause testsuite failures dougsemler at gmail dot com
2010-05-16 14:23 ` [Bug testsuite/44159] " dougsemler at gmail dot com
2010-05-17 14:55 ` ktietz at gcc dot gnu dot org
2010-06-07 10:57 ` ktietz at gcc dot gnu dot org
2010-06-07 11:09 ` ktietz at gcc dot gnu dot org
2010-06-07 11:09 ` ktietz at gcc dot gnu dot org

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).