Re: [PATCHES] equal() perf tweak

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: Postgresql Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [PATCHES] equal() perf tweak
Date: 2003-11-04 18:51:41
Message-ID: 3FA7F53D.30709@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

Tom Lane wrote:

>Andrew Dunstan <andrew(at)dunslane(dot)net> writes:
>
>
>>Just a thought - if we are messing with the List definition should we at
>>the same time address the strict aliasing issues arising from Node's
>>multiple personalities (I think it is the main offender).
>>
>>
>
>
>
>>Or is the intention never to do this, or not any time soon?
>>
>>
>
>I have no intention of messing with the Node concept; it's built into
>the backend far too firmly to consider any significant change.
>
>I don't think we understand exactly what we'd have to avoid in order to
>enable strict aliasing, but if it requires getting rid of Node then it
>ain't happening. (I doubt that it does, anyway. I think the issues are
>probably quite localized. The main problem I see is that we don't have
>any trustworthy check to find out everyplace that strict aliasing could
>cause problems.)
>
>
>

*nod* (I made the last point previously :-) )

I don't claim that my understanding is anything like complete.

However, in the case of gcc it might be easier to fix than you think,
without a lot of declaration rearranging, using gcc's new "may_alias"
type attribute, (see
http://gcc.gnu.org/onlinedocs/gcc-3.3.2/gcc/Type-Attributes.html ) which
could probably be added in a very few places (e.g. Node's typedef) to
fix things.

Of course, that doesn't help when vendor foo's compiler starts doing
type-based alias analysis.

Still, to quote "Airplane", that's not important right now.

cheers

andrew

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Gaetano Mendola 2003-11-04 18:55:39 Re: 7.4RC1 tag'd, branched and bundled ...
Previous Message Greg Stark 2003-11-04 18:28:58 Re: Experimental patch for inter-page delay in VACUUM

Browse pgsql-patches by date

  From Date Subject
Next Message Peter Eisentraut 2003-11-05 00:14:32 Re: UW 713UP3 patch
Previous Message Tom Lane 2003-11-04 17:31:21 Re: [PATCHES] equal() perf tweak