diff mbox

[boehm-gc,build] Don't link libgcjgc with libdl on IRIX

Message ID yddzkqer1b9.fsf@manam.CeBiTec.Uni-Bielefeld.DE
State New
Headers show

Commit Message

Rainer Orth Feb. 2, 2011, 1:52 p.m. UTC
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 mbox

Patch

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")