public inbox for java-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [boehm-gc, build] Don't link libgcjgc with libdl on IRIX
@ 2011-02-02 13:52 Rainer Orth
  0 siblings, 0 replies; only message in thread
From: Rainer Orth @ 2011-02-02 13:52 UTC (permalink / raw)
  To: gcc-patches; +Cc: java-patches

As mentioned in

	[testsuite, build] Convert boehm-gc testsuite to DejaGnu (PR boehm-gc/11412)
        http://gcc.gnu.org/ml/gcc-patches/2011-01/msg00244.html
        http://gcc.gnu.org/ml/gcc-patches/2011-01/msg01128.html
        http://gcc.gnu.org/ml/gcc-patches/2011-01/msg01638.html

I noticed while there exists an N32 (and O32) libdl on IRIX, it is
missing for the N64 ABI.  This causes problems for the boehm-gc
testsuite, since this is currently autoconfigured and stored in
testsuite/site.exp, but DejaGnu only uses site.exp for the default
multilib.  While this could be worked around either in the testsuite
itself or by emitting Tcl code into site.exp which detects the current
multilib, the issue can be avoided completely by noting that libdl isn't
necessary at all since dlopen and friends already live in libc.

This is what this patch does.  Bootstrapped without regressions on
mips-sgi-irix6.5.  Now the new boehm-gc testsuite also passes for the
N64 multilib (with one unrelated exception which I'm still
investigating).

Installed.

	Rainer


2011-01-28  Rainer Orth  <ro@CeBiTec.Uni-Bielefeld.DE>

	* configure.ac: Don't use libdl on mips-sgi-irix6*.
	* configure: Regenerate.

diff -r 6b3ce68ea0da boehm-gc/configure.ac
--- a/boehm-gc/configure.ac	Fri Jan 28 13:02:03 2011 +0100
+++ b/boehm-gc/configure.ac	Fri Jan 28 13:05:44 2011 +0100
@@ -1,4 +1,4 @@
-# Copyright (c) 1999, 2000, 2001, 2002, 2003, 2006, 2010 by Red Hat, Inc.
+# Copyright (c) 1999, 2000, 2001, 2002, 2003, 2006, 2010, 2011 by Red Hat, Inc.
 # All rights reserved.
 # Copyright 2004 Nathanael Nerode
 # 
@@ -312,9 +312,12 @@
   *) ;;
 esac
 
-# We never want libdl on darwin. It is a fake libdl that just ends up making
-# dyld calls anyway
 case "$host" in
+  # While IRIX 6 has libdl for the O32 and N32 ABIs, it's missing for N64
+  # and unnecessary everywhere.
+  mips-sgi-irix6*) ;;
+  # We never want libdl on darwin. It is a fake libdl that just ends up making
+  # dyld calls anyway
   *-*-darwin*) ;;
   *)
     AC_CHECK_LIB(dl, dlopen, EXTRA_TEST_LIBS="$EXTRA_TEST_LIBS -ldl")


-- 
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2011-02-02 13:52 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-02-02 13:52 [boehm-gc, build] Don't link libgcjgc with libdl on IRIX Rainer Orth

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).