Re: pgsql: Use special gcc -W flags only if we are using

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: pgsql-committers(at)postgresql(dot)org
Subject: Re: pgsql: Use special gcc -W flags only if we are using
Date: 2006-04-22 02:09:28
Message-ID: 200604220209.k3M29SQ26799@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Tom Lane wrote:
> momjian(at)postgresql(dot)org (Bruce Momjian) writes:
> > Use special gcc -W flags only if we are using gcc, not if we are using
> > the Intel compiler.
>
> I'm not entirely thrilled with this patch, because it disables not only
> the -W options but also -fno-strict-aliasing. Are you sure the latter
> is not needed for ICC?

Peter asked for specific problems with the warnings, and the submitter
produced results. I see no mention of -fno-strict-aliasing:

http://archives.postgresql.org/pgsql-patches/2006-04/msg00002.php

However, the -f flag is outside the block he modified:

# Disable strict-aliasing rules; needed for gcc 3.3+
PGAC_PROG_CC_CFLAGS_OPT([-fno-strict-aliasing])

That flag is still used for both gcc and the intel compiler, and because
he tested the patch, I assume it works for both too.

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

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

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2006-04-22 02:27:33 Re: pgsql: Use special gcc -W flags only if we are using gcc, not if we are
Previous Message Tom Lane 2006-04-22 01:57:31 Re: pgsql: Use special gcc -W flags only if we are using gcc, not if we are