From c5257345ef61922468cd9abd887c3cb6c38792cb Mon Sep 17 00:00:00 2001 From: Andres Freund Date: Mon, 15 Oct 2018 15:24:33 -0700 Subject: Move TupleTableSlots boolean member into one flag variable. There's several reasons for this change: 1) It reduces the total size of TupleTableSlot / reduces alignment padding, making the commonly accessed members fit into a single cacheline (but we currently do not force proper alignment, so that's not yet guaranteed to be helpful) 2) Combining the booleans into a flag allows to combine read/writes from memory. 3) With the upcoming slot abstraction changes, it allows to have core and extended flags, in a memory efficient way. Author: Ashutosh Bapat and Andres Freund Discussion: https://postgr.es/m/20180220224318.gw4oe5jadhpmcdnm@alap3.anarazel.de --- src/backend/jit/llvm/llvmjit_expr.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'src/backend/jit/llvm/llvmjit_expr.c') diff --git a/src/backend/jit/llvm/llvmjit_expr.c b/src/backend/jit/llvm/llvmjit_expr.c index 99e0cee157f..e5fe116acbc 100644 --- a/src/backend/jit/llvm/llvmjit_expr.c +++ b/src/backend/jit/llvm/llvmjit_expr.c @@ -292,7 +292,7 @@ llvm_compile_expr(ExprState *state) if (!desc && is && is->ps_ResultTupleSlot && - is->ps_ResultTupleSlot->tts_fixedTupleDescriptor) + TTS_FIXED(is->ps_ResultTupleSlot)) desc = is->ps_ResultTupleSlot->tts_tupleDescriptor; } else if (opcode == EEOP_OUTER_FETCHSOME) @@ -304,7 +304,7 @@ llvm_compile_expr(ExprState *state) if (!desc && os && os->ps_ResultTupleSlot && - os->ps_ResultTupleSlot->tts_fixedTupleDescriptor) + TTS_FIXED(os->ps_ResultTupleSlot)) desc = os->ps_ResultTupleSlot->tts_tupleDescriptor; } else -- cgit v1.2.3