pgsql: Move EEOP_*_SYSVAR evaluation out of line.

From: Andres Freund <andres(at)anarazel(dot)de>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Move EEOP_*_SYSVAR evaluation out of line.
Date: 2018-11-07 19:16:12
Message-ID: E1gKTJE-00038h-Mc@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Move EEOP_*_SYSVAR evaluation out of line.

This mainly de-duplicates code. As evaluating a system variable isn't
the hottest path and the current inline implementation ends up calling
out to an external function anyway, this is OK from a performance POV.

The main motivation for de-duplicating is the upcoming slot
abstraction work, after which there's not guaranteed to be a HeapTuple
backing the slot.

Author: Andres Freund, Amit Khandekar
Discussion: https://postgr.es/m/20181105210039.hh4vvi4vwoq5ba2q@alap3.anarazel.de

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/b84a6dafbf2bb921baee53c0c1aba7719ee38817

Modified Files
--------------
src/backend/executor/execExprInterp.c | 58 ++++++++++++-----------------------
src/backend/jit/llvm/llvmjit.c | 4 +--
src/backend/jit/llvm/llvmjit_expr.c | 38 +++++------------------
src/backend/jit/llvm/llvmjit_types.c | 2 +-
src/include/executor/execExpr.h | 2 ++
src/include/jit/llvmjit.h | 2 +-
6 files changed, 33 insertions(+), 73 deletions(-)

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Andrew Dunstan 2018-11-07 19:18:28 pgsql: Use parallel installcheck in vcregress.pl's upgrade test
Previous Message Andrew Dunstan 2018-11-07 17:59:44 pgsql: Quiet valgrind complaints following pread/pwrite changes