[Bug testsuite/47954] New: gcc.dg/tree-ssa/ssa-ccp-33.c fails with link error on m68k-linux

[Bug testsuite/47954] New: gcc.dg/tree-ssa/ssa-ccp-33.c fails with link error on m68k-linux

[mikpe at it dot uu.se]

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

           Summary: gcc.dg/tree-ssa/ssa-ccp-33.c fails with link error on
                    m68k-linux
           Product: gcc
           Version: 4.6.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: testsuite
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: mikpe@it.uu.se


On m68k-linux the gcc.dg/tree-ssa/ssa-ccp-33.c test case fails:

> fgrep ssa-ccp-33 gcc/testsuite/gcc/gcc.log
Executing on host: /mnt/scratch/objdir46/gcc/xgcc -B/mnt/scratch/objdir46/gcc/
/mnt/scratch/gcc-4.6-20110226/gcc/testsuite/gcc.dg/tree-ssa/ssa-ccp-33.c   -O 
-lm   -o ssa-ccp-33.exe    (timeout = 300)
ssa-ccp-33.c:(.text+0x2a): undefined reference to `link_error'
ssa-ccp-33.c:(.text+0x2a): undefined reference to `link_error'
FAIL: gcc.dg/tree-ssa/ssa-ccp-33.c (test for excess errors)
ssa-ccp-33.c:(.text+0x2a): undefined reference to `link_error'

The test case uses "sizeof(int)-1" to generate a mask of bits that must be zero
in an "int *p", and calls link_error() if this isn't the case, the intention
being that the compiler completely eliminates the call.

However, on m68k-linux __alignof__(int) == 2 while sizeof(int) == 4, so the
compiler fails to optimize the check away, leading to the link error.

The test case should use __alignof__ not sizeof.

[Bug testsuite/47954] gcc.dg/tree-ssa/ssa-ccp-33.c fails with link error on m68k-linux

[rguenth at gcc dot gnu.org]

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

Richard Guenther <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2011.03.02 13:40:10
     Ever Confirmed|0                           |1

--- Comment #1 from Richard Guenther <rguenth at gcc dot gnu.org> 2011-03-02 13:40:10 UTC ---
A patch changing it that way is pre-approved.

[Bug testsuite/47954] gcc.dg/tree-ssa/ssa-ccp-33.c fails with link error on m68k-linux

[rguenth at gcc dot gnu.org]

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

--- Comment #2 from Richard Guenther <rguenth at gcc dot gnu.org> 2011-03-08 10:45:49 UTC ---
Author: rguenth
Date: Tue Mar  8 10:45:46 2011
New Revision: 170773

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=170773
Log:
2011-03-08  Mikael Pettersson  <mikpe@it.uu.se>

    PR testsuite/47954
    * gcc.dg/tree-ssa/ssa-ccp-33.c: Use __alignof__ not
    sizeof to compute alignment.

Modified:
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/testsuite/gcc.dg/tree-ssa/ssa-ccp-33.c

[Bug testsuite/47954] gcc.dg/tree-ssa/ssa-ccp-33.c fails with link error on m68k-linux

[mikpe at it dot uu.se]

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

Mikael Pettersson <mikpe at it dot uu.se> changed:

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

--- Comment #3 from Mikael Pettersson <mikpe at it dot uu.se> 2011-03-08 14:13:15 UTC ---
Fixed.