public inbox for [email protected]  
help / color / mirror / Atom feed
From: Daniel Gustafsson <[email protected]>
To: [email protected]
Subject: pgsql: Replace EEOP_DONE with special steps for return/no return
Date: Tue, 11 Mar 2025 11:10:44 +0000
Message-ID: <[email protected]> (raw)

Replace EEOP_DONE with special steps for return/no return

Knowing when the side-effects of an expression is the intended result
of the execution, rather than the returnvalue, is important for being
able generate more efficient JITed code. This replaces EEOP_DONE with
two new steps: EEOP_DONE_RETURN and EEOP_DONE_NO_RETURN.  Expressions
which return a value should use the former step; expressions used for
their side-effects which don't return value should use the latter.

Author: Andres Freund <[email protected]>
Co-authored-by: Daniel Gustafsson <[email protected]>
Reviewed-by: Andreas Karlsson <[email protected]>
Discussion: https://postgr.es/m/[email protected]
Discussion: https://postgr.es/m/[email protected]

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/8dd7c7cd0a2605d5301266a6b67a569d6a305106

Modified Files
--------------
src/backend/executor/README           | 11 ++++++--
src/backend/executor/execExpr.c       | 24 ++++++++--------
src/backend/executor/execExprInterp.c | 25 +++++++++++------
src/backend/executor/nodeAgg.c        |  7 ++---
src/backend/jit/llvm/llvmjit_expr.c   |  6 +++-
src/include/executor/execExpr.h       |  7 +++--
src/include/executor/executor.h       | 52 +++++++++++++++++++++++++++++++++--
7 files changed, 98 insertions(+), 34 deletions(-)



reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected]
  Subject: Re: pgsql: Replace EEOP_DONE with special steps for return/no return
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox