[Patch,build] PR 46540 - add --disable-libquadmath/--disable-libquadmath-support

[Tobias Burnus]

Tobias Burnus wrote:
> H.J. Lu wrote:
>> On Sat, Dec 4, 2010 at 8:15 AM, Tobias Burnus<burnus@net-b.de>  wrote:
>>> This patch adds two configure options:
>>> --disable-libquadmath
>>>     which disables the libquadmath build (e.g. to use the system's
>>> libquadmath)
>> This isn't very clear.  We already have --with-system-zlib, why not
>> --with-system-libquadmath?

Well, we also have: --enable-libada and --enable-libssp.

I have now used --with-system-libquadmath but I have to admit I find it 
much more misleading. --disable-libquadmath does that the option was 
stating: It does not build "libquadmath".

--with-system-libquadmath somehow implies that the system *has* to 
provide libquadmath - which is not (generally) true. It is just not 
build. -- Only if you build the Fortran front end on a system which 
supports __float128 and you did not disable the support using 
--disable-libquadmath-support, you have to provide libquadmath such that 
it is in the include ("-I") and library ("-L") path.

For the case you only build, e.g., C or C++ the option 
--without-system-libquadmath implies that the library is build and 
--with-system-libquadmath (default on systems supporting it) implies 
that the library is not build (note: a system library is not needed) -- 
which I find much more confusing. Or if I want to build on x86-64 
Fortran without quadmath support and without the library, the options 
you have to use are: --disable-libquadmath-support 
--with-system-libquadmath -- which somehow looks strange.


The attached patch has been a couple of times in different variants 
bootstrapped on x86-64-linux.
OK for the trunk?

(For the variant with --disable-libquadmath, cf. patch snippet below.)

Tobias

PS: I have included a patch for "gcc/doc/install.html" (describing 
--disable-libquadmath[-support]).

PPS: A variant would for --(disable|enable)-libquadmath:

+AC_ARG_ENABLE(libquadmath,
+[  --disable-libquadmath   do not build libquadmath directory],
+ENABLE_LIBQUADMATH=$enableval,
+ENABLE_LIBQUADMATH=yes)
+if test "${ENABLE_LIBQUADMATH}" = "no" ; then
+  noconfigdirs="$noconfigdirs target-libquadmath"
+fi
+
+

-------------- next part --------------
A non-text attachment was scrubbed...
Name: quad-disable-v2.diff
Type: text/x-patch
Size: 4544 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/fortran/attachments/20101205/4ddad808/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: install.diff
Type: text/x-patch
Size: 967 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/fortran/attachments/20101205/4ddad808/attachment-0001.bin>