[Bug testsuite/40532] New: FAIL: gcc.dg/builtins-65.c (test for excess errors)

[Bug testsuite/40532] New: FAIL: gcc.dg/builtins-65.c (test for excess errors)

[danglin at gcc dot gnu dot org]

Executing on host: /test/gnu/gcc/objdir/gcc/xgcc -B/test/gnu/gcc/objdir/gcc/
/te
st/gnu/gcc/gcc/gcc/testsuite/gcc.dg/builtins-65.c   -O2 -ffast-math  -lm   -o
bu
iltins-65.exe    (timeout = 300)
/usr/ccs/bin/ld: Unsatisfied symbols:
   link_error (first referenced in /var/tmp//ccyLWKiQ.o) (code)
   logbf (first referenced in /var/tmp//ccyLWKiQ.o) (code)
   logbl (first referenced in /var/tmp//ccyLWKiQ.o) (code)
   ilogbf (first referenced in /var/tmp//ccyLWKiQ.o) (code)
   ilogbl (first referenced in /var/tmp//ccyLWKiQ.o) (code)
collect2: ld returned 1 exit status
compiler exited with status 1
output is:
/usr/ccs/bin/ld: Unsatisfied symbols:
   link_error (first referenced in /var/tmp//ccyLWKiQ.o) (code)
   logbf (first referenced in /var/tmp//ccyLWKiQ.o) (code)
   logbl (first referenced in /var/tmp//ccyLWKiQ.o) (code)
   ilogbf (first referenced in /var/tmp//ccyLWKiQ.o) (code)
   ilogbl (first referenced in /var/tmp//ccyLWKiQ.o) (code)
collect2: ld returned 1 exit status

FAIL: gcc.dg/builtins-65.c (test for excess errors)


-- 
           Summary: FAIL: gcc.dg/builtins-65.c (test for excess errors)
           Product: gcc
           Version: 4.5.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: testsuite
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: danglin at gcc dot gnu dot org
 GCC build triplet: hppa2.0w-hp-hpux11.11
  GCC host triplet: hppa2.0w-hp-hpux11.11
GCC target triplet: hppa2.0w-hp-hpux11.11


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

[Bug testsuite/40532] FAIL: gcc.dg/builtins-65.c (test for excess errors)

[ubizjak at gmail dot com]

------- Comment #1 from ubizjak at gmail dot com  2009-06-24 06:35 -------
Can you put HAVE_C99_RUNTIME around problematic conversions (just copy the
approach from builtins-18.c) ?


-- 


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

[Bug testsuite/40532] FAIL: gcc.dg/builtins-65.c (test for excess errors)

[dave at hiauly1 dot hia dot nrc dot ca]

------- Comment #2 from dave at hiauly1 dot hia dot nrc dot ca  2009-06-25 00:32 -------
Subject: Re:  FAIL: gcc.dg/builtins-65.c (test for
        excess errors)

> Can you put HAVE_C99_RUNTIME around problematic conversions (just copy the
> approach from builtins-18.c) ?

Attached diff.  However, there's still a call left to linK_error.
Attached builtins-65.c.133t.optimized.

Dave


------- Comment #3 from dave at hiauly1 dot hia dot nrc dot ca  2009-06-25 00:32 -------
Created an attachment (id=18065)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=18065&action=view)


------- Comment #4 from dave at hiauly1 dot hia dot nrc dot ca  2009-06-25 00:32 -------
Created an attachment (id=18066)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=18066&action=view)


-- 


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

[Bug testsuite/40532] FAIL: gcc.dg/builtins-65.c (test for excess errors)

[ubizjak at gmail dot com]

------- Comment #5 from ubizjak at gmail dot com  2009-06-25 07:26 -------
(In reply to comment #2)

> > Can you put HAVE_C99_RUNTIME around problematic conversions (just copy the
> > approach from builtins-18.c) ?
> 
> Attached diff.  However, there's still a call left to linK_error.


This is due to the fact that your libm provides logb and ilogb.

However, according to linux mapages, these are C99 functions. Your target
doesn't define TARGET_C99_FUNCTIONS, so optimizer does not recognize logb and
ilogb as functions that can be converted.

I will simply disable builtins-65.c for non-C99 targets


-- 

ubizjak at gmail dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |ubizjak at gmail dot com
                   |dot org                     |
             Status|UNCONFIRMED                 |ASSIGNED
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2009-06-25 07:26:05
               date|                            |


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

[Bug testsuite/40532] FAIL: gcc.dg/builtins-65.c (test for excess errors)

[ubizjak at gmail dot com]

------- Comment #6 from ubizjak at gmail dot com  2009-06-25 07:33 -------
(In reply to comment #5)

> I will simply disable builtins-65.c for non-C99 targets

... like this:

Index: builtins-65.c
===================================================================
--- builtins-65.c       (revision 148916)
+++ builtins-65.c       (working copy)
@@ -1,5 +1,6 @@
 /* { dg-do link } */
 /* { dg-options "-O2 -ffast-math" } */
+/* { dg-require-effective-target c99_runtime } */

 extern int ilogbf (float);
 extern float logbf (float);


-- 


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

[Bug testsuite/40532] FAIL: gcc.dg/builtins-65.c (test for excess errors)

[uros at gcc dot gnu dot org]

------- Comment #7 from uros at gcc dot gnu dot org  2009-06-26 09:02 -------
Subject: Bug 40532

Author: uros
Date: Fri Jun 26 09:02:04 2009
New Revision: 148967

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=148967
Log:
        PR testsuite/40532
        * gcc.dg/builtins-65.c: Require c99 runtime.


Modified:
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/testsuite/gcc.dg/builtins-65.c


-- 


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

[Bug testsuite/40532] FAIL: gcc.dg/builtins-65.c (test for excess errors)

[ubizjak at gmail dot com]

------- Comment #8 from ubizjak at gmail dot com  2009-06-26 09:04 -------
Fixed.


-- 

ubizjak at gmail dot com changed:

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


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