From: | Andres Freund <andres(at)anarazel(dot)de> |
---|---|
To: | pgsql-committers(at)lists(dot)postgresql(dot)org |
Subject: | pgsql: Prevent generating EEOP_AGG_STRICT_INPUT_CHECK operations when n |
Date: | 2018-11-03 23:08:25 |
Message-ID: | E1gJ51l-0002si-Sw@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Prevent generating EEOP_AGG_STRICT_INPUT_CHECK operations when nargs == 0.
This only became a problem with 4c640f4f38, which didn't synchronize
the value agg_strict_input_check.nargs is set to, with the guard
condition for emitting the operation.
Besides such instructions being unnecessary overhead, currently the
LLVM JIT provider doesn't support them. It seems more sensible to
avoid generating such instruction than supporting them. Add assertions
to make it easier to debug a potential further occurance.
Discussion: https://postgr.es/m/2a505161-2727-2473-7c46-591ed108ac52@email.cz
Backpatch: 11-, like 4c640f4f38.
Branch
------
REL_11_STABLE
Details
-------
https://git.postgresql.org/pg/commitdiff/6eb31cedb316d59bf9c63658868ff0fcddcc085a
Modified Files
--------------
src/backend/executor/execExpr.c | 2 +-
src/backend/jit/llvm/llvmjit_expr.c | 2 ++
2 files changed, 3 insertions(+), 1 deletion(-)
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2018-11-04 16:21:22 | pgsql: Fix unused-variable warning. |
Previous Message | Andres Freund | 2018-11-03 23:08:24 | pgsql: Prevent generating EEOP_AGG_STRICT_INPUT_CHECK operations when n |