[Bug bootstrap/46451] New: legacy cloog-ppl test omits necessary linkage on ppl libraries

[Bug bootstrap/46451] New: legacy cloog-ppl test omits necessary linkage on ppl libraries

[howarth at nitro dot med.uc.edu]

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

           Summary: legacy cloog-ppl test omits necessary linkage on ppl
                    libraries
           Product: gcc
           Version: 4.6.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: bootstrap
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: howarth@nitro.med.uc.edu


The changes in...

Author: spop
Date: Thu Nov 11 21:33:24 2010
New Revision: 166622

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=166622
Log:
Support official CLooG.org versions.

2010-11-11  Andreas Simbuerger  <simbuerg@fim.uni-passau.de>

    * configure.ac: Support official CLooG.org versions.
    * configure: Regenerate.
    * config/cloog.m4: New.

Added:
    trunk/config/cloog.m4
Modified:
    trunk/ChangeLog
    trunk/configure
    trunk/configure.ac

...break the bootstrap when using...


../gcc-4.6-20101112/configure --prefix=/sw --prefix=/sw/lib/gcc4.6
--mandir=/sw/share/man --infodir=/sw/lib/gcc4.6/info
--enable-languages=c,c++,fortran,objc,obj-c++,java --with-gmp=/sw
--with-libiconv-prefix=/sw --with-ppl=/sw --with-cloog=/sw --with-mpc=/sw
--with-system-zlib --x-includes=/usr/X11R6/include --x-libraries=/usr/X11R6/lib
--program-suffix=-fsf-4.6 --enable-checking=yes

on x86_64-apple-darwin10. This is because the test for legacy cloog expects
access to the _ppl_version_major symbol but neglects to append ${ppllibs} onto
the LDFLAGS for that test...


configure:5735: checking for installed CLooG
configure:5752: gcc -o conftest -g -O2 -I/sw/include -I/sw/include 
-I/sw/include -I/sw/include   -DCLOOG_PPL_BACKEND -DCLOOG_INT_GMP -DCLOOG_ORG 
-L/sw/lib conftest.c -lcloog  >&5
Undefined symbols:
  "_ppl_version_major", referenced from:
      _main in ccILDtHX.o
ld: symbol(s) not found
collect2: ld returned 1 exit status
configure:5752: $? = 1
configure: failed program was:
| /* confdefs.h */
| #define PACKAGE_NAME ""
| #define PACKAGE_TARNAME ""
| #define PACKAGE_VERSION ""
| #define PACKAGE_STRING ""
| #define PACKAGE_BUGREPORT ""
| #define PACKAGE_URL ""
| #define LT_OBJDIR ".libs/"
| /* end confdefs.h.  */
| #include <cloog/cloog.h>
| int
| main ()
| {
| ppl_version_major ()
|   ;
|   return 0;
| }

[Bug bootstrap/46451] legacy cloog-ppl test omits necessary linkage on ppl libraries

[howarth at nitro dot med.uc.edu]

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

--- Comment #1 from Jack Howarth <howarth at nitro dot med.uc.edu> 2010-11-12 15:55:57 UTC ---
Fixing this seems problematic. While config/cloog.m4 currently has the location
of the ppl headers passed to CFLAGS for these tests...

  dnl clooglibs & clooginc may have been initialized by CLOOG_INIT_FLAGS.
  CFLAGS="${CFLAGS} ${clooginc} ${pplinc} ${gmpinc}"
  CPPFLAGS="${CPPFLAGS} ${_clooglegacyinc} ${_cloogorginc}"
  LDFLAGS="${LDFLAGS} ${clooglibs}"

...the LDFLAGS entry omits ${pplibs}. Adding ${pplibs} to LDFLAGS would fix the
legacy cloog test but would result in an undesired linkage on ppl for the
cloog-isl test. This section should be reworked to only pass ${pplinc} to
CFLAGS and ${pplibs} to LDFLAGS for only the legacy cloog and cloog-ppl tests.

[Bug bootstrap/46451] legacy cloog-ppl test omits necessary linkage on ppl libraries

[howarth at nitro dot med.uc.edu]

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

--- Comment #2 from Jack Howarth <howarth at nitro dot med.uc.edu> 2010-11-12 19:44:01 UTC ---
I can confirm that...

Author: grosser
Date: Fri Nov 12 18:29:27 2010
New Revision: 166669

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=166669
Log:
Pass PPL libraries to CLooG version check

    * config/cloog.m4: Pass ppl libraries to the CLooG version check.
    * configure: Regenerate.

Modified:
    trunk/ChangeLog
    trunk/config/cloog.m4
    trunk/configure

Author: grosser
Date: Fri Nov 12 18:29:37 2010
New Revision: 166670

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=166670
Log:
Fix typo

    * config/cloog.m4: Fix typo.  verison -> version.
    * configure: Regenerate.

Modified:
    trunk/ChangeLog
    trunk/config/cloog.m4
    trunk/configure

eliminated the problems with configure detecting the presence of legacy cloog
support.

[Bug bootstrap/46451] legacy cloog-ppl test omits necessary linkage on ppl libraries

[howarth at nitro dot med.uc.edu]

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

Jack Howarth <howarth at nitro dot med.uc.edu> changed:

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

--- Comment #3 from Jack Howarth <howarth at nitro dot med.uc.edu> 2010-11-22 00:05:41 UTC ---
Fixed at r166669/166670.