Re: nbtree: Cache operator family OID in _bt_setup_array_cmp

From: Peter Geoghegan <pg(at)bowt(dot)ie>
To: Peter Eisentraut <peter(at)eisentraut(dot)org>
Cc: Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com>, Kirill Reshke <reshkekirill(at)gmail(dot)com>, Postgres hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: nbtree: Cache operator family OID in _bt_setup_array_cmp
Date: 2026-01-07 15:32:51
Message-ID: CAH2-WzkcsHx7mh5FYzXSwNNg3vv0GcFX5+rCfkaxuh2CNSTuug@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, Jan 7, 2026 at 9:45 AM Peter Eisentraut <peter(at)eisentraut(dot)org> wrote:
> Are you claiming that this has a performance impact, or that it makes
> the code easier to understand, or something else? The patch isn't
> necessarily wrong, but a clear description of the motivation would be good.

I've noticed that making a local copy of a variable can sometimes help
in tight inner loops, by avoiding an aliasing issue. But it is only
something I've seen help in extreme, rare cases.

--
Peter Geoghegan

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Fujii Masao 2026-01-07 16:14:19 Re: Remove deprecated role membership options from psql help for CREATE USER/GROUP
Previous Message Matheus Alcantara 2026-01-07 15:17:38 Re: Import Statistics in postgres_fdw before resorting to sampling.