Re[2]: [Gc] Re: Build failure on Solaris 10 with Sun Studio

Ivan Maidanski ivmai at mail.ru
Sat Sep 11 04:29:44 PDT 2010


Hi!

Fixed (and regenerated).

Fri, 10 Sep 2010 00:22:43 +0200 ludo at gnu.org (Ludovic CourtХs):

> Hi Ivan,
> Ivan Maidanski <ivmai at mail.ru> writes:
> > Ludo - Hey, don't be pessimistic!
> >
> > We could add a check if defined(inline) to gc_priv.h to make
> > AC_C_INLINE working. If it's not difficult, please provide a patch
> for
> > GC thinking as if we always use configure (and I'll try to find the
> > solution acceptable to both parties).
> Indeed, I was too pessimistic.  ;-)
> Here’s a patch that should meet the criteria.
> Thanks,
> Ludo’.
> 2010-09-10  Ludovic Courtès <ludo at gnu.org>
> * configure.ac: Use `AC_C_INLINE'.
> * include/private/gc_priv.h (GC_INLINE)[HAVE_CONFIG_H]: Use the
> `inline' keyword determined by `AC_C_INLINE'.
> Index: configure.ac
> ===================================================================
> RCS file: /cvsroot/bdwgc/bdwgc/configure.ac,v
> retrieving revision 1.61
> diff -u -r1.61 configure.ac
> --- configure.ac	27 Jul 2010 21:30:18 -0000	1.61
> +++ configure.ac	9 Sep 2010 22:20:49 -0000
> @@ -96,6 +96,9 @@
> AH_TEMPLATE([_POSIX_C_SOURCE], [The POSIX feature macro.])
> AH_TEMPLATE([_PTHREADS], [Indicates the use of pthreads (NetBSD).])
> +dnl Check for a flavor of supported `inline' keyword.
> +AC_C_INLINE
> +
> THREADDLLIBS=
> need_atomic_ops_asm=false
> ## Libraries needed to support dynamic loading and/or threads.
> Index: include/private/gc_priv.h
> ===================================================================
> RCS file: /cvsroot/bdwgc/bdwgc/include/private/gc_priv.h,v
> retrieving revision 1.78
> diff -u -r1.78 gc_priv.h
> --- include/private/gc_priv.h	19 Mar 2010 19:21:02 -0000	1.78
> +++ include/private/gc_priv.h	9 Sep 2010 22:20:50 -0000
> @@ -137,7 +137,10 @@
> # define EXPECT(expr, outcome) (expr)
> #endif /* __GNUC__ */
> -#if defined(_MSC_VER) || defined(__INTEL_COMPILER) || defined(__DMC__) \
> +#ifdef HAVE_CONFIG_H
> +/* The `inline' keyword as determined by Autoconf's
> `AC_C_INLINE'.  */
> +# define GC_INLINE static inline
> +#elif defined(_MSC_VER) || defined(__INTEL_COMPILER) || defined(__DMC__) \
> || defined(__WATCOMC__)
> # define GC_INLINE static __inline
> #elif (__GNUC__ >= 3) || defined(__sun)
> ATTACHMENT: application/pgp-signature


More information about the Gc mailing list