[Bug fortran/46520] New: libquadmath: Build also with --enable-languages=c; fails with some cross targets

[Bug fortran/46520] New: libquadmath: Build also with --enable-languages=c; fails with some cross targets

[burnus at gcc dot gnu.org]

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

           Summary: libquadmath: Build also with --enable-languages=c;
                    fails with some cross targets
           Product: gcc
           Version: 4.6.0
            Status: UNCONFIRMED
          Keywords: build
          Severity: normal
          Priority: P3
         Component: fortran
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: burnus@gcc.gnu.org


Seemingly, libquadmath is also build with  --enable-languages=c  only;
currently, it shouldn't.

Possibly solved by:

--- gcc/fortran/config-lang.in  (revision 166859)
+++ gcc/fortran/config-lang.in  (working copy)
@@ -30 +30 @@ compilers="f951\$(exeext)"
-target_libs=target-libgfortran
+target_libs=target-libquadmath target-libgfortran



Additionally, it fails to configure at some link test with
  --enable-languages=c --target=moxie-elf
which is a pure target board.

Ian wrote in #gcc:
  according to antgreen, the libquadmath configure script expects to
  be able to link an executable

[Bug fortran/46520] [4.6 Regression] libquadmath: Build also with --enable-languages=c; fails with some cross targets

[jsm28 at gcc dot gnu.org]

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

Joseph S. Myers <jsm28 at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.6.0
            Summary|libquadmath: Build also     |[4.6 Regression]
                   |with --enable-languages=c;  |libquadmath: Build also
                   |fails with some cross       |with --enable-languages=c;
                   |targets                     |fails with some cross
                   |                            |targets

--- Comment #1 from Joseph S. Myers <jsm28 at gcc dot gnu.org> 2010-11-22 18:21:55 UTC ---
The failure to build for bare-metal targets that can't link executables at
configure test time is a regression.  I see this on arm-none-eabi.

[Bug fortran/46520] [4.6 Regression] libquadmath: Build also with --enable-languages=c; fails with some cross targets

[pinskia at gcc dot gnu.org]

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

Andrew Pinski <pinskia at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |tschwinge at gcc dot
                   |                            |gnu.org

--- Comment #2 from Andrew Pinski <pinskia at gcc dot gnu.org> 2010-11-22 20:42:50 UTC ---
*** Bug 46612 has been marked as a duplicate of this bug. ***

[Bug fortran/46520] [4.6 Regression] libquadmath: Build also with --enable-languages=c; fails with some cross targets

[burnus at gcc dot gnu.org]

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

--- Comment #3 from Tobias Burnus <burnus at gcc dot gnu.org> 2010-11-23 23:02:43 UTC ---
Cf. partially complimentary, partially overlapping patches at
http://gcc.gnu.org/ml/fortran/2010-11/msg00291.html and
http://gcc.gnu.org/ml/gcc-patches/2010-11/msg02189.html

[Bug fortran/46520] [4.6 Regression] libquadmath: Build also with --enable-languages=c; fails with some cross targets

[burnus at gcc dot gnu.org]

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

--- Comment #4 from Tobias Burnus <burnus at gcc dot gnu.org> 2010-12-08 19:52:01 UTC ---
For __float128 detection, there is now a patch at:
  http://gcc.gnu.org/ml/fortran/2010-12/msg00045.html

However, one still might change the configure check for symbol versioning.

As there have been already several commits, which also affect cross building:
Can someone confirm that building GCC on a bare iron machine still fails due to
the libquadmath configure script?

[Bug fortran/46520] [4.6 Regression] libquadmath: Build also with --enable-languages=c; fails with some cross targets

[burnus at gcc dot gnu.org]

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

--- Comment #5 from Tobias Burnus <burnus at gcc dot gnu.org> 2010-12-08 21:04:50 UTC ---
Author: burnus
Date: Wed Dec  8 21:04:45 2010
New Revision: 167605

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=167605
Log:
2010-12-08  Tobias Burnus  <burnus@net-b.de>

        PR fortran/46520
        * configure.ac: Use GCC_TRY_COMPILE_OR_LINK instead of
        AC_TRY_LINK and cache libquad_have_float128.
        * configure: Regenerate.


Modified:
    trunk/libquadmath/ChangeLog
    trunk/libquadmath/configure
    trunk/libquadmath/configure.ac

[Bug fortran/46520] [4.6 Regression] libquadmath: fails at link test on bare irons

[burnus at gcc dot gnu.org]

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

Tobias Burnus <burnus at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|[4.6 Regression]            |[4.6 Regression]
                   |libquadmath: Build also     |libquadmath: fails at link
                   |with --enable-languages=c;  |test on bare irons
                   |fails with some cross       |
                   |targets                     |

--- Comment #6 from Tobias Burnus <burnus at gcc dot gnu.org> 2010-12-14 20:19:20 UTC ---
Current result for picochip-unknown-none:

checking how to run the C preprocessor... [...]
checking for sqrtl in -lm... configure: error: Link tests are not allowed after
GCC_NO_EXECUTABLES.
make[2]: *** [configure-target-libquadmath] Error 1

Draft patch - might be insufficient:

--- a/libquadmath/configure.ac
+++ b/libquadmath/configure.ac
@@ -109,8 +109,10 @@ esac
 AC_SUBST(toolexecdir)
 AC_SUBST(toolexeclibdir)

-AC_CHECK_LIB([m],[sqrtl],[AC_DEFINE([HAVE_SQRTL],[1],[libm includes sqrtl])])
-AC_CHECK_LIB([m],[cbrtl],[AC_DEFINE([HAVE_CBRTL],[1],[libm includes cbrtl])])
+if test x$gcc_no_link != xyes; then
+  AC_CHECK_LIB([m],[sqrtl],[AC_DEFINE([HAVE_SQRTL],[1],[libm includes
sqrtl])])
+  AC_CHECK_LIB([m],[cbrtl],[AC_DEFINE([HAVE_CBRTL],[1],[libm includes
cbrtl])])
+fi

 # Check for symbol versioning (copied from libssp).
 AC_MSG_CHECKING([whether symbol versioning is supported])

[Bug fortran/46520] [4.6 Regression] libquadmath: fails at link test on bare irons

[burnus at gcc dot gnu.org]

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

--- Comment #7 from Tobias Burnus <burnus at gcc dot gnu.org> 2010-12-19 19:01:41 UTC ---
Author: burnus
Date: Sun Dec 19 19:01:38 2010
New Revision: 168069

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=168069
Log:
2010-12-19  Tobias Burnus  <burnus@net-b.de>

        PR fortran/46520
        * configure.ac: Do not call AC_CHECK_LIB for gcc_no_link.
        * configure: Regenerate


Modified:
    trunk/libquadmath/ChangeLog
    trunk/libquadmath/configure
    trunk/libquadmath/configure.ac

[Bug fortran/46520] [4.6 Regression] libquadmath: fails at link test on bare irons

[dfranke at gcc dot gnu.org]

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

Daniel Franke <dfranke at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |WAITING
   Last reconfirmed|                            |2010.12.27 00:33:08
                 CC|                            |dfranke at gcc dot gnu.org
     Ever Confirmed|0                           |1

--- Comment #8 from Daniel Franke <dfranke at gcc dot gnu.org> 2010-12-27 00:33:08 UTC ---
Tobias, anything left here or can this report be closed?

[Bug fortran/46520] [4.6 Regression] libquadmath: fails at link test on bare irons

[burnus at gcc dot gnu.org]

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

Tobias Burnus <burnus at gcc dot gnu.org> changed:

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

--- Comment #9 from Tobias Burnus <burnus at gcc dot gnu.org> 2011-01-05 08:57:59 UTC ---
(In reply to comment #8)
> Tobias, anything left here or can this report be closed?

I was just told for picochip-unknown-none:

> I just tried a build and libquadmath works fine. I had to
> disable libiberty, though.


Thus, I close this PR as FIXED. If someone still has problems, please report
them and either re-open this PR or fill a new PR (and CC me).