diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2006-04-29 20:47:31 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2006-04-29 20:47:31 +0000 |
| commit | f0df09678531530835deceb6df652d23549bfc4e (patch) | |
| tree | 2fe4d59330d5e5592b8e82b17cc5e1a52803c859 /configure.in | |
| parent | de762468aa7258b8974174462ccb25f05a95f591 (diff) | |
| download | postgresql-f0df09678531530835deceb6df652d23549bfc4e.tar.gz | |
Rearrange some configure.in comments for better readability.
Commit configure and pg_config.h.in, missed in last configure.in
update.
Diffstat (limited to 'configure.in')
| -rw-r--r-- | configure.in | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/configure.in b/configure.in index 83a2906259..a6162e9235 100644 --- a/configure.in +++ b/configure.in @@ -1,5 +1,5 @@ dnl Process this file with autoconf to produce a configure script. -dnl $PostgreSQL: pgsql/configure.in,v 1.463 2006/04/29 16:34:41 momjian Exp $ +dnl $PostgreSQL: pgsql/configure.in,v 1.464 2006/04/29 20:47:29 tgl Exp $ dnl dnl Developers, please strive to achieve this order: dnl @@ -248,19 +248,20 @@ else fi fi -# The Intel compiler on Linux supports most gcc options, but not -# all of them, so only add these options if we are really using -# gcc. +# Some versions of GCC support some additional useful warning flags. +# Check whether they are supported, and add them to CFLAGS if so. if test "$GCC" = yes; then + +# ICC pretends to be GCC but it's lying; it doesn't support these options. +# So we have to check if "GCC" is really ICC. AC_TRY_COMPILE([], [@%:@ifndef __INTEL_COMPILER choke me @%:@endif], [ICC=[yes]], [ICC=[no]]) + if test "$ICC" = no; then CFLAGS="$CFLAGS -Wall -Wmissing-prototypes -Wpointer-arith -Winline" - # Some versions of GCC support some additional useful warning flags. - # Check whether they are supported, and add them to CFLAGS if so. PGAC_PROG_CC_CFLAGS_OPT([-Wdeclaration-after-statement]) PGAC_PROG_CC_CFLAGS_OPT([-Wendif-labels]) else |
