instrumentation: Move ExecProcNodeInstr to allow inlining This moves the implementation of ExecProcNodeInstr, the ExecProcNode variant that gets used when instrumentation is on, to be defined in instrument.c instead of execProcNode.c, and marks functions it uses as inline.
This allows compilers to generate an optimized implementation, and shows a 4 to 12% reduction in instrumentation overhead for queries that move lots of rows. Author: Lukas Fittl <[email protected]> Suggested-by: Andres Freund <[email protected]> Reviewed-by: Andres Freund <[email protected]> Discussion: https://postgr.es/m/cap53pkzdbk8vj1fs4az481lgmn8f9mjic39zrhqkfusyq6k...@mail.gmail.com Branch ------ master Details ------- https://git.postgresql.org/pg/commitdiff/544000288ec8f7dc6a1e0285821adc47324ecd33 Modified Files -------------- src/backend/executor/execProcnode.c | 20 -------------------- src/backend/executor/instrument.c | 37 +++++++++++++++++++++++++++++++------ src/include/executor/executor.h | 7 +++++++ 3 files changed, 38 insertions(+), 26 deletions(-)
