Re: pgsql: Fix compiler builtin usage in new pg_bitutils.c

From: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: Re: pgsql: Fix compiler builtin usage in new pg_bitutils.c
Date: 2019-02-15 20:18:05
Message-ID: 20190215201804.GA791@alvherre.pgsql
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

On 2019-Feb-15, Tom Lane wrote:

> Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> writes:
> > I'm done with this stuff. Anybody feel free to run with it, but for a
> > barely noticeable performance improvement it's not going to be me.
>
> Sure, I'll pick it up. I agree it's probably a marginal performance
> change, but it seems a shame to give up when we were 80% of the way
> there.

(BTW, my reading of the articles I cited, as well as my own runs of the
test programs therein, suggest that in order to get a really good
performance improvement you need to hand-code calls to the POPCNT
instruction in assembly rather than rely on the compiler intrinsics. I
think there is^Wwas almost enough infrastructure to do that. I'm not
sure that the operations being optimized by these changes are
interesting enough, in the grand scheme of things.)

--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2019-02-15 20:23:17 Re: pgsql: Fix compiler builtin usage in new pg_bitutils.c
Previous Message Tom Lane 2019-02-15 19:57:29 Re: pgsql: Fix compiler builtin usage in new pg_bitutils.c