Re: [HACKERS] 8.3.3 compiler warnings with gcc 4.3

2008-09-15 Thread Gregory Stark
Tom Lane [EMAIL PROTECTED] writes:

 Devrim =?ISO-8859-1?Q?G=DCND=DCZ?= [EMAIL PROTECTED] writes:
 I'm getting these on Fedora-9:
 tqual.c:115: warning: inlining failed in call to ‘SetHintBits’: call is

 They're just cosmetic.  We don't generally worry about fixing cosmetic
 warnings in back branches.

Are they? It seems like these were marked inline for a reason. SetHintBits has
at least one conditional in it which can often be optimized out (it's either
InvalidTransactionId or else has been tested already).

In the tuplestore case there's not much point in having that function if it's
not going to be inlined. I'm imagining that it was put in because someone did
verify that the non-inlined version was consuming significant time. -- Perhaps
a bit assumption though.

Also, is 8.3 really a back branch? It's the current release and anyone
downloading Postgres from source on the web site will be getting these
warnings -- and there are a lot of them. Enough that it looks like something's
gone wrong with the build.

-- 
  Gregory Stark
  EnterpriseDB  http://www.enterprisedb.com
  Ask me about EnterpriseDB's RemoteDBA services!

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


[HACKERS] 8.3.3 compiler warnings with gcc 4.3

2008-09-13 Thread Devrim GÜNDÜZ
I'm getting these on Fedora-9:

tqual.c:115: warning: inlining failed in call to ‘SetHintBits’: call is
unlikely and code size would grow
tqual.c:377: warning: called from here

tuplesort.c: In function ‘comparetup_index’:
tuplesort.c:2423: warning: inlining failed in call to ‘myFunctionCall2’:
--param large-stack-frame-growth limit reached
tuplesort.c:2474: warning: called from here

All these appear a few more times during compilation.

I'm not sure we fixed these in 8.3 branch, and wanted to post it
-- 
Devrim GÜNDÜZ, RHCE
devrim~gunduz.org, devrim~PostgreSQL.org, devrim.gunduz~linux.org.tr
   http://www.gunduz.org


signature.asc
Description: This is a digitally signed message part


Re: [HACKERS] 8.3.3 compiler warnings with gcc 4.3

2008-09-13 Thread Tom Lane
Devrim =?ISO-8859-1?Q?G=DCND=DCZ?= [EMAIL PROTECTED] writes:
 I'm getting these on Fedora-9:
 tqual.c:115: warning: inlining failed in call to ‘SetHintBits’: call is

They're just cosmetic.  We don't generally worry about fixing cosmetic
warnings in back branches.

regards, tom lane

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers