Re: patch to have configure check if CC is intel C compiler - Mailing list pgsql-patches

From Bruce Momjian
Subject Re: patch to have configure check if CC is intel C compiler
Date
Msg-id 200604212311.k3LNBgN27708@candle.pha.pa.us
Whole thread Raw
In response to patch to have configure check if CC is intel C compiler  (Jeremy Drake <pgsql-patches@jdrake.com>)
Responses Re: patch to have configure check if CC is intel C compiler  (Jeremy Drake <pgsql-patches@jdrake.com>)
List pgsql-patches
Comment added and patch applied.  Thanks.

---------------------------------------------------------------------------


Jeremy Drake wrote:
> If configure sees that the compiler specified by $CC looks like gcc
> (defines __GNUC__), then it puts some extra command line options into the
> CFLAGS (mostly -W*).  The intel C compiler for linux emulates gcc by
> default, which means it defines that and looks very much like gcc to
> configure.  However, it does not get along with the added -W flags very
> well.  They don't seem to kill it, but some of them give warnings about
> unsupported command line options and others produce insane amounts of
> output from the compiler.
>
> This patch makes configure check for the __INTEL_COMPILER define (which is
> the recommended way to detect the intel compiler) before adding the extra
> CFLAGS if it thinks the compiler is GCC.  I am not an autoconf hacker, so
> I may have done it wrong or something, but it appears to work for me (ICC
> 9.0.032 on gentoo i686 with latest packages).
>
> The patch is against the HEAD but I think it should be similar for other
> branches (I have been compiling 8.0 and 8.1 for some time with the intel
> compiler by manually massaging the makefiles after configure ran).
>
>
> --
> We the unwilling, led by the ungrateful, are doing the impossible.
> We've done so much, for so long, with so little,
> that we are now qualified to do something with nothing.

Content-Description:

[ Attachment, skipping... ]

>
> ---------------------------(end of broadcast)---------------------------
> TIP 3: Have you checked our extensive FAQ?
>
>                http://www.postgresql.org/docs/faq

--
  Bruce Momjian   http://candle.pha.pa.us
  EnterpriseDB    http://www.enterprisedb.com

  + If your life is a hard drive, Christ can be your backup. +

pgsql-patches by date:

Previous
From: "Magnus Hagander"
Date:
Subject: Re: Win32 semaphore patch
Next
From: Bruce Momjian
Date:
Subject: Re: restore to defaults values when commenting of variables