[Bug target/22083] New: TARGET_C99_FUNCTIONS is wrongly defined on AIX 5.1

[Bug target/22083] New: TARGET_C99_FUNCTIONS is wrongly defined on AIX 5.1

[lmakhlis at bmc dot com]

gcc/config/rs6000/aix51.h and gcc/config/rs6000/aix52.h both define
TARGET_C99_FUNCTIONS.  This seems to be wrong for AIX 5.1: it has xxxl
functions, but not xxxf ones.

Source code:
=== CUT ===
extern double sqrt(double);

float f(float x)
{
  return sqrt(x);
}

int main()
{
  return f(1.0f) != 1.0f;
}
=== CUT ===

"gcc -v" output:

=== CUT ===
Reading specs from /bmc/moe/lib/gcc/powerpc-ibm-aix5.1.0.0/3.4.3/specs
Configured with: /depot/gnu/gcc-3.4.3/configure --prefix=/bmc/moe
--srcdir=/depot/gnu/gcc-3.4.3 --exec-prefix=/bmc/moe
--with-local-prefix=/bmc/moe --enable-shared --with-as=/usr/bin/as
--with-ld=/usr/bin/ld --enable-threads=posix --with-x --enable-java-awt=xlib
--disable-nls
Thread model: aix
gcc version 3.4.3
=== CUT ===

"oslevel -r" output:
5100-04

"lslpp -l bos.adt.libm" output:
=== CUT ===
  Fileset                      Level  State      Description
  ----------------------------------------------------------------------------
Path: /usr/lib/objrepos
  bos.adt.libm              5.1.0.25  COMMITTED  Base Application Development
                                                 Math Library
=== CUT ===

"gcc <file> -O -lm" output:
ld: 0711-317 ERROR: Undefined symbol: .sqrtf
ld: 0711-345 Use the -bloadmap or -bnoquiet option to obtain more information.
collect2: ld returned 8 exit status

The problem becomes even worse for C++ code:
libstdc++ correctly detects that (e.g.) sqrtf() is missing, and generates a stub
that calls sqrt().  But the compiler converts that sqrt() call back into
sqrtf(), resulting in infinite recursion!  The following example triggers this:

=== CUT ===
#include <cmath>

int main()
{
  return std::sqrt(1.0f) != 1.0f;
}
=== CUT ===

-- 
           Summary: TARGET_C99_FUNCTIONS is wrongly defined on AIX 5.1
           Product: gcc
           Version: 3.4.3
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: target
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: lmakhlis at bmc dot com
                CC: gcc-bugs at gcc dot gnu dot org
 GCC build triplet: powerpc-ibm-aix5.1.0.0
  GCC host triplet: powerpc-ibm-aix5.1.0.0
GCC target triplet: powerpc-ibm-aix5.1.0.0


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

[Bug target/22083] TARGET_C99_FUNCTIONS is wrongly defined on AIX 5.1

[dje at gcc dot gnu dot org]

------- Additional Comments From dje at gcc dot gnu dot org  2005-06-17 13:56 -------
The macro was defined for AIX 5.1 by Roger Sayle:

2003-07-04  Roger Sayle  <roger@eyesopen.com>

        * config/rs6000/aix51.h (TARGET_C99_FUNCTIONS): Define.
        * config/rs6000/aix52.h (TARGET_C99_FUNCTIONS): Likewise.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |roger at eyesopen dot com
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|                            |1
   Last reconfirmed|0000-00-00 00:00:00         |2005-06-17 13:56:51
               date|                            |


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

[Bug target/22083] TARGET_C99_FUNCTIONS is wrongly defined on AIX 5.1

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2005-06-17 14:02 -------
See <http://gcc.gnu.org/ml/gcc-patches/2003-07/msg00389.html> for the patch which caused it.

-- 


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

[Bug target/22083] [3.4/4.0/4.1 Regression] TARGET_C99_FUNCTIONS is wrongly defined on AIX 5.1

[pinskia at gcc dot gnu dot org]

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |wrong-code
            Summary|TARGET_C99_FUNCTIONS is     |[3.4/4.0/4.1 Regression]
                   |wrongly defined on AIX 5.1  |TARGET_C99_FUNCTIONS is
                   |                            |wrongly defined on AIX 5.1
   Target Milestone|---                         |3.4.5


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

[Bug target/22083] [3.4/4.0/4.1 Regression] TARGET_C99_FUNCTIONS is wrongly defined on AIX 5.1

[roger at eyesopen dot com]

------- Additional Comments From roger at eyesopen dot com  2005-06-17 17:22 -------
Unfortunately, the AIX 5.1 machine that was loaned to OpenEye by IBM has had to
be returned since this patch was submitted/applied in 2003.  So my only guess is
that this may have been a patch level issue with bos.adt.libm as I'm fairly
confident that our 5.1 machine had sqrtf and friends when the patch was
developed.  We do still have a 5.2 machine, and that certainly has sqrtf.
Could this be related to the version(s) of IBM's Visual Age C/C++, that was
installed on the machine?

Unfortunately, without access to a 5.1 box I'm unable to properly test the
obvious trivial change to config/rs6000/aix51.h.  Clearly, if there are flavors
of AIX 5.1 out there without sqrtf, it unsafe to define TARGET_C99_FUNCTIONS in
aix51.h.  Fortunately, such a change won't negatively affect users of AIX 5.2 or
later.

Dave, I'm happy to post an untested patch to gcc-patches if that's appropriate.
Alternatively, Lev could you try removing the five lines I added to aix51.h and
confirm that that resolves the issues you're seeing.

Many thanks in advance.  Sorry for any inconvenience.


-- 


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

[Bug target/22083] [3.4/4.0/4.1 Regression] TARGET_C99_FUNCTIONS is wrongly defined on AIX 5.1

[lmakhlis at bmc dot com]

------- Additional Comments From lmakhlis at bmc dot com  2005-06-17 19:01 -------
It might be an issue with the patch level of libm -- I was testing on 5.1 ML04,
which was released in 2002 -- but I've found another machine at ML05 (released
in Oct 2003), and it still doesn't have sqrtf and friends in libm.  I don't have
access to anything newer than that right now.

I doubt it's related to Visual Age. VAC's implementation of the standard C++
library, /usr/lib/libC.a, does have sqrtf and friends -- but I'm not linking
with it when using GCC.

Before submitting the bug report, I tried removing the macro definition in
aix51.h, and it does solve the problem.  There is a related C++-specific problem
that still isn't solved, but maybe that merits a separate PR.  The problem is:

If you have C++ code that uses std::sqrt(float) and build a dynamically linked
executable on AIX 5.1, then bring it over to AIX 5.2, it won't run, because
sqrtf() is now in libm, but not in libstdc++.  This doesn't seem to be a problem
on other platforms (for example, a program compiled on Solaris 8, against
sqrtf() in libstdc++, runs fine on Solaris 10, with sqrtf() in libm), but on AIX
the dynamic loader looks for a symbol only in a specific library.  I am guessing
it's a difference between COFF and ELF?

As a side point, TARGET_C99_FUNCTION isn't defined on any Solaris platforms.  I
think it's okay to define it on Solaris 9 and up?

-- 


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

[Bug target/22083] [3.4/4.0/4.1 Regression] TARGET_C99_FUNCTIONS is wrongly defined on AIX 5.1

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2005-06-17 19:05 -------
(In reply to comment #4)
> If you have C++ code that uses std::sqrt(float) and build a dynamically linked
> executable on AIX 5.1, then bring it over to AIX 5.2, it won't run, because
> sqrtf() is now in libm, but not in libstdc++.  This doesn't seem to be a problem
> on other platforms (for example, a program compiled on Solaris 8, against
> sqrtf() in libstdc++, runs fine on Solaris 10, with sqrtf() in libm), but on AIX
> the dynamic loader looks for a symbol only in a specific library.  I am guessing
> it's a difference between COFF and ELF?

That is not a bug, you cannot do that as libstdc++ is not forward compatiable, this is also happens on 
Darwin (Mac OS X).

-- 


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

[Bug target/22083] [3.4/4.0/4.1 Regression] TARGET_C99_FUNCTIONS is wrongly defined on AIX 5.1

[ebotcazou at gcc dot gnu dot org]

------- Additional Comments From ebotcazou at gcc dot gnu dot org  2005-06-17 19:13 -------
> As a side point, TARGET_C99_FUNCTION isn't defined on any Solaris platforms.  I
> think it's okay to define it on Solaris 9 and up?

Only Solaris 10 has full C99 math support out of the box (and after a bit of
tweaking to make it usable for GCC).

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


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

[Bug target/22083] [3.4/4.0/4.1 Regression] TARGET_C99_FUNCTIONS is wrongly defined on AIX 5.1

[lmakhlis at bmc dot com]

------- Additional Comments From lmakhlis at bmc dot com  2005-06-17 19:32 -------
Andrew: I am trying to run older (5.1) programs on a newer (5.2) system, not
vice versa -- why would that require forward (rather than backward) compatibility?

-- 


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

[Bug target/22083] [3.4/4.0/4.1 Regression] TARGET_C99_FUNCTIONS is wrongly defined on AIX 5.1

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2005-06-18 13:26 -------
Subject: Bug 22083

CVSROOT:	/cvs/gcc
Module name:	gcc
Changes by:	sayle@gcc.gnu.org	2005-06-18 13:26:20

Modified files:
	gcc            : ChangeLog 
	gcc/config/rs6000: aix51.h 

Log message:
	PR target/22083
	* config/rs6000/aix51.h (TARGET_C99_FUNCTIONS): Remove definition.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.diff?cvsroot=gcc&r1=2.9180&r2=2.9181
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/rs6000/aix51.h.diff?cvsroot=gcc&r1=1.28&r2=1.29



-- 


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

[Bug target/22083] [3.4/4.0 Regression] TARGET_C99_FUNCTIONS is wrongly defined on AIX 5.1

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2005-06-30 21:54 -------
*** Bug 22261 has been marked as a duplicate of this bug. ***

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |bugzilla-gcc at
                   |                            |thewrittenword dot com


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

[Bug target/22083] [3.4/4.0 Regression] TARGET_C99_FUNCTIONS is wrongly defined on AIX 5.1

[mmitchel at gcc dot gnu dot org]

------- Additional Comments From mmitchel at gcc dot gnu dot org  2005-07-06 16:28 -------
Reset target milestone to 4.0.2.

(Gaby and I have a standing agreement that the more current release branch gets
the target milestone.)

Patch OK for 4.0.2.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|3.4.5                       |4.0.2


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

[Bug target/22083] [3.4/4.0 Regression] TARGET_C99_FUNCTIONS is wrongly defined on AIX 5.1

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2005-07-08 18:28 -------
Subject: Bug 22083

CVSROOT:	/cvs/gcc
Module name:	gcc
Branch: 	gcc-4_0-branch
Changes by:	dje@gcc.gnu.org	2005-07-08 18:28:25

Modified files:
	gcc            : ChangeLog 
	gcc/config/rs6000: aix51.h 

Log message:
	2005-07-08  David Edelsohn  <edelsohn@gnu.org>
	
	Backport from mainline:
	2005-06-18  Roger Sayle  <roger@eyesopen.com>
	PR target/22083
	* config/rs6000/aix51.h (TARGET_C99_FUNCTIONS): Remove definition.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.diff?cvsroot=gcc&only_with_tag=gcc-4_0-branch&r1=2.7592.2.301&r2=2.7592.2.302
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/rs6000/aix51.h.diff?cvsroot=gcc&only_with_tag=gcc-4_0-branch&r1=1.25&r2=1.25.8.1



-- 


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

[Bug target/22083] [3.4/4.0 Regression] TARGET_C99_FUNCTIONS is wrongly defined on AIX 5.1

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2005-07-08 18:33 -------
Subject: Bug 22083

CVSROOT:	/cvs/gcc
Module name:	gcc
Branch: 	gcc-3_4-branch
Changes by:	dje@gcc.gnu.org	2005-07-08 18:33:45

Modified files:
	gcc            : ChangeLog 
	gcc/config/rs6000: aix51.h 

Log message:
	2005-07-08  David Edelsohn  <edelsohn@gnu.org>
	
	Backport from mainline:
	2005-06-18  Roger Sayle  <roger@eyesopen.com>
	PR target/22083
	* config/rs6000/aix51.h (TARGET_C99_FUNCTIONS): Remove definition.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=2.2326.2.879&r2=2.2326.2.880
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/rs6000/aix51.h.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.20&r2=1.20.14.1



-- 


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

[Bug target/22083] [3.4/4.0 Regression] TARGET_C99_FUNCTIONS is wrongly defined on AIX 5.1

[dje at gcc dot gnu dot org]

------- Additional Comments From dje at gcc dot gnu dot org  2005-07-08 18:35 -------
Patch applied to gcc-4.0 and gcc-3.4 branches.

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


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

[Bug target/22083] [3.4/4.0 Regression] TARGET_C99_FUNCTIONS is wrongly defined on AIX 5.1

[pinskia at gcc dot gnu dot org]

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
      Known to work|3.3.3 4.1.0                 |3.3.3 4.1.0 4.0.2
   Target Milestone|4.0.2                       |3.4.5


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