Re: [PATCH v4] Avoid manual shift-and-test logic in AllocSetFreeIndex - Mailing list pgsql-hackers

From Tom Lane
Subject Re: [PATCH v4] Avoid manual shift-and-test logic in AllocSetFreeIndex
Date
Msg-id 11828.1248185002@sss.pgh.pa.us
Whole thread Raw
In response to Re: [PATCH v4] Avoid manual shift-and-test logic in AllocSetFreeIndex  (Jeremy Kerr <jk@ozlabs.org>)
List pgsql-hackers
Jeremy Kerr <jk@ozlabs.org> writes:
> Thanks for the benchmark app, thought I'd pitch in with some ppc 
> results:

It looks to me like we should go with the lookup table approach, as
being the best tradeoff of speed improvement vs platform and compiler
independence.  The "float hack" is right out ;-)

I wonder whether it is worth fooling with alternatives for the data type
of the lookup table entries.  unsigned char might be a tad faster
(avoid useless sign-extension work).  int might be faster yet, but it
would enlarge the table, creating a distributed overhead that the
microbenchmark would completely fail to show.  Or we could buy that
back by reducing the table to cover only 6 bits, knowing that 12 is
more than we need.
        regards, tom lane


pgsql-hackers by date:

Previous
From: Magnus Hagander
Date:
Subject: Re: [PATCH] user mapping extension to pg_ident.conf
Next
From: Tom Lane
Date:
Subject: Re: [PATCH] user mapping extension to pg_ident.conf