From fe2e206cdb00a2d5dcebc3c8e8119017392d9781 Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Sun, 5 Jul 2020 13:12:31 -0400 Subject: [PATCH] Inline the fast path of plpgsql's exec_cast_value(). In the common case where this function isn't actually asked to perform any type conversion, there's nothing it has to do beyond comparing the arguments. Arrange for that part to be inlined into callers, with the slower path remaining out-of-line. This seems to be good for several percent speedup on simple cases, with only minimal code bloat. Amit Khandekar Discussion: https://postgr.es/m/CAJ3gD9eBNrmUD7WBBLG8ohaZ485H9y+4eihQTgr+K8Lhka3vcQ@mail.gmail.com --- src/pl/plpgsql/src/pl_exec.c | 65 ++++++++++++++++++++++++------------ 1 file changed, 43 insertions(+), 22 deletions(-) diff --git a/src/pl/plpgsql/src/pl_exec.c b/src/pl/plpgsql/src/pl_exec.c index 54900e01c8f..d4a3d58daa9 100644 --- a/src/pl/plpgsql/src/pl_exec.c +++ b/src/pl/plpgsql/src/pl_exec.c @@ -417,10 +417,14 @@ static void instantiate_empty_record_variable(PLpgSQL_execstate *estate, PLpgSQL_rec *rec); static char *convert_value_to_string(PLpgSQL_execstate *estate, Datum value, Oid valtype); -static Datum exec_cast_value(PLpgSQL_execstate *estate, - Datum value, bool *isnull, - Oid valtype, int32 valtypmod, - Oid reqtype, int32 reqtypmod); +static inline Datum exec_cast_value(PLpgSQL_execstate *estate, + Datum value, bool *isnull, + Oid valtype, int32 valtypmod, + Oid reqtype, int32 reqtypmod); +static Datum do_cast_value(PLpgSQL_execstate *estate, + Datum value, bool *isnull, + Oid valtype, int32 valtypmod, + Oid reqtype, int32 reqtypmod); static plpgsql_CastHashEntry *get_cast_hashentry(PLpgSQL_execstate *estate, Oid srctype, int32 srctypmod, Oid dsttype, int32 dsttypmod); @@ -7825,7 +7829,7 @@ convert_value_to_string(PLpgSQL_execstate *estate, Datum value, Oid valtype) * done with the result. * ---------- */ -static Datum +static inline Datum exec_cast_value(PLpgSQL_execstate *estate, Datum value, bool *isnull, Oid valtype, int32 valtypmod, @@ -7837,30 +7841,47 @@ exec_cast_value(PLpgSQL_execstate *estate, if (valtype != reqtype || (valtypmod != reqtypmod && reqtypmod != -1)) { - plpgsql_CastHashEntry *cast_entry; + /* We keep the slow path out-of-line. */ + value = do_cast_value(estate, value, isnull, valtype, valtypmod, + reqtype, reqtypmod); + } - cast_entry = get_cast_hashentry(estate, - valtype, valtypmod, - reqtype, reqtypmod); - if (cast_entry) - { - ExprContext *econtext = estate->eval_econtext; - MemoryContext oldcontext; + return value; +} - oldcontext = MemoryContextSwitchTo(get_eval_mcontext(estate)); +/* ---------- + * do_cast_value Slow path for exec_cast_value. + * ---------- + */ +static Datum +do_cast_value(PLpgSQL_execstate *estate, + Datum value, bool *isnull, + Oid valtype, int32 valtypmod, + Oid reqtype, int32 reqtypmod) +{ + plpgsql_CastHashEntry *cast_entry; - econtext->caseValue_datum = value; - econtext->caseValue_isNull = *isnull; + cast_entry = get_cast_hashentry(estate, + valtype, valtypmod, + reqtype, reqtypmod); + if (cast_entry) + { + ExprContext *econtext = estate->eval_econtext; + MemoryContext oldcontext; - cast_entry->cast_in_use = true; + oldcontext = MemoryContextSwitchTo(get_eval_mcontext(estate)); - value = ExecEvalExpr(cast_entry->cast_exprstate, econtext, - isnull); + econtext->caseValue_datum = value; + econtext->caseValue_isNull = *isnull; - cast_entry->cast_in_use = false; + cast_entry->cast_in_use = true; - MemoryContextSwitchTo(oldcontext); - } + value = ExecEvalExpr(cast_entry->cast_exprstate, econtext, + isnull); + + cast_entry->cast_in_use = false; + + MemoryContextSwitchTo(oldcontext); } return value; -- 2.30.2