This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Add libquadmath - and use it in gfortran (round SIX)


On Mon, Nov 15, 2010 at 10:10:40AM +0100, Tobias Burnus wrote:
> Dear all, dear Richard and dear Ian,
>
> The patch has been reviewed by the brand-new build-maintainer Ralf  
> (build part), by Steve (Fortran part), includes suggestions and fixes by  
> Joseph, Jakub, Ralf, Steve and others. It has been bootstraped and  
> regtested on x86-64-Linux, x86-64-Darwin, x86-64 and i686 FreeBSD and  
> Cygwin.
>
> Thus, I think it is time for a review of a global maintainer.
>
> The patch consists of
>
> a) The libquadmath library itself, which is licensed under the LGPL:   
> http://gcc.gnu.org/ml/gcc-patches/2010-11/msg00790/libquadmath.tar.bz2
>
> b) A patch to change its configure.ac to use "subdir-objects": Attached.
>
> c) The patch to build and to use libquadmath in gfortran and  
> libgfortran: Attached.
>
> d) The changelog at http://gcc.gnu.org/ml/gcc-patches/2010-11/txt00112.txt
>
> e) A test case at http://gcc.gnu.org/ml/gcc-patches/2010-11/txt00070.txt
>
>
> Changes since round FIVE  
> (http://gcc.gnu.org/ml/gcc-patches/2010-11/msg01442.html):
>
> * Addition of patch (b): Suggested change by Ralf at  
> http://gcc.gnu.org/ml/gcc-patches/2010-11/msg01477.html
> * Patch (c) contains the changes for Ralf's review comment at  
> http://gcc.gnu.org/ml/gcc-patches/2010-11/msg01473.html
>
> (As those changes are minor, I have not included an update of the  
> generated files; you can find those for round FOUR at   
> http://gcc.gnu.org/ml/gcc-patches/2010-11/msg01442/quad-gen.diff.bz2 )
>
> Is the patch OK for the trunk?
>
> Tobias


> --- /dev/shm/libquadmath/configure.ac	2010-11-08 20:58:50.000000000 +0100
> +++ libquadmath/configure.ac	2010-11-14 20:08:23.000000000 +0100
> @@ -23,10 +23,10 @@ AC_CONFIG_HEADER(config.h)
>  AC_CANONICAL_SYSTEM
>  target_alias=${target_alias-$host_alias}
>  
> -AM_INIT_AUTOMAKE([1.9.0 foreign no-dist -Wall -Werror])
> +AM_INIT_AUTOMAKE([1.9.0 foreign subdir-objects no-dist -Wall -Werror])
>  
>  AC_PROG_CC
> -
> +AM_PROG_CC_C_O
>  
>  AC_MSG_CHECKING([for --enable-version-specific-runtime-libs])
>  AC_ARG_ENABLE(version-specific-runtime-libs,


Tobias,
  How did you regenerate configure in libquadmath with this change? Using autoconf 2.64,
I keep getting...

configure.ac:29: error: possibly undefined macro: AM_PROG_CC_C_O
      If this token and others are legitimate, please use m4_pattern_allow.
      See the Autoconf documentation.

            Jack


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]