On Mon, Jan 18, 2010 at 10:10 AM, Brian Paul <bri...@vmware.com> wrote: > Alan Coopersmith wrote: >> Signed-off-by: Alan Coopersmith <alan.coopersm...@sun.com> >> --- >> configure.ac | 9 ++++++++- >> 1 files changed, 8 insertions(+), 1 deletions(-) >> >> diff --git a/configure.ac b/configure.ac >> index d01d0f1..f121002 100644 >> --- a/configure.ac >> +++ b/configure.ac >> @@ -96,7 +96,14 @@ esac >> >> dnl Add flags for gcc and g++ >> if test "x$GCC" = xyes; then >> - CFLAGS="$CFLAGS -Wall -Wmissing-prototypes -std=c99 -ffast-math >> -fvisibility=hidden" >> + CFLAGS="$CFLAGS -Wall -Wmissing-prototypes -std=c99 -ffast-math" >> + >> + # Enable -fvisibility=hidden if using a gcc that supports it >> + save_CFLAGS="$CFLAGS" >> + AC_MSG_CHECKING([whether $(CC) supports -fvisibility=hidden]) >> + CFLAGS="$CFLAGS -fvisibility=hidden" >> + AC_LINK_IFELSE([AC_LANG_PROGRAM()], AC_MSG_RESULT([yes]), >> + [CFLAGS="$save_CFLAGS" ; AC_MSG_RESULT([no])]); >> >> # Work around aliasing bugs - developers should comment this out >> CFLAGS="$CFLAGS -fno-strict-aliasing" > > I'm not the configure guy, but this looks OK to me.
Nice and clean, although I didn't know that AC_LANG_PROGRAM with no argument would work. Signed-off-by: Dan Nicholson <dbn.li...@gmail.com> ------------------------------------------------------------------------------ Throughout its 18-year history, RSA Conference consistently attracts the world's best and brightest in the field, creating opportunities for Conference attendees to learn about information security's most important issues through interactions with peers, luminaries and emerging and established companies. http://p.sf.net/sfu/rsaconf-dev2dev _______________________________________________ Mesa3d-dev mailing list Mesa3d-dev@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mesa3d-dev
