Re: [PATCH] PR bootstrap/52623 Disable libquadmath on AIX

2012-04-03 Thread David Edelsohn
On Mon, Mar 26, 2012 at 7:40 AM, Joseph S. Myers
 wrote:
> On Fri, 23 Mar 2012, David Edelsohn wrote:
>
>> The build process of libquadmath sometimes encounters problems on AIX
>> due to multilib and LD_LIBRARY_PATH interfering with GCC's own library
>> dependencies.  Libquadmath is not used on AIX, so this patch adds it
>> to noconfigdirs.
>
> Please include this explanation in a comment on the powerpc-*-aix* |
> rs6000-*-aix* case.  Existing directory-disabling cases may be
> uncommented, but new ones ought to have comments explaining the reason for
> disabling the library.

Is the following okay?

Thanks, David

Index: configure.ac
===
--- configure.ac(revision 186108)
+++ configure.ac(working copy)
@@ -509,6 +509,15 @@
 fi
 fi

+# Disable libquadmath for some systems.
+case "${target}" in
+  # libquadmath is unused on AIX and libquadmath build process use of
+  # LD_LIBRARY_PATH can break AIX bootstrap.
+  powerpc-*-aix* | rs6000-*-aix*)
+noconfigdirs="$noconfigdirs target-libquadmath"
+;;
+esac
+
 # Disable libssp for some systems.
 case "${target}" in
   avr-*-*)


Re: [PATCH] PR bootstrap/52623 Disable libquadmath on AIX

2012-03-26 Thread Joseph S. Myers
On Fri, 23 Mar 2012, David Edelsohn wrote:

> The build process of libquadmath sometimes encounters problems on AIX
> due to multilib and LD_LIBRARY_PATH interfering with GCC's own library
> dependencies.  Libquadmath is not used on AIX, so this patch adds it
> to noconfigdirs.

Please include this explanation in a comment on the powerpc-*-aix* | 
rs6000-*-aix* case.  Existing directory-disabling cases may be 
uncommented, but new ones ought to have comments explaining the reason for 
disabling the library.

-- 
Joseph S. Myers
jos...@codesourcery.com