Commit1307bc3

committed

Reset context at the tail end of JITed EEOP_AGG_PLAIN_TRANS.

While no negative consequences are currently known, it's clearly wrongto not reset the context in one of the branches.Reported-By: Dmitry DolgovAuthor: Dmitry DolgovDiscussion:https://postgr.es/m/CAGPqQf165-=+Drw3Voim7M5EjHT1zwPF9BQRjLFQzCzYnNZEiQ@mail.gmail.comBackpatch: 11-, where JIT compilation support was added

1 parent013f320 commit1307bc3Copy full SHA for 1307bc3

File tree

-0

lines changed

-0

lines changed

Lines changed: 2 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -2499,6 +2499,8 @@ llvm_compile_expr(ExprState *state)`
`2499`	`2499`	`/* store trans value */`
`2500`	`2500`	`LLVMBuildStore(b,v_newval,v_transvaluep);`
`2501`	`2501`	`LLVMBuildStore(b,v_fcinfo_isnull,v_transnullp);`
	`2502`	`+`
	`2503`	`+l_mcxt_switch(mod,b,v_oldcontext);`
`2502`	`2504`	`LLVMBuildBr(b,opblocks[i+1]);`
`2503`	`2505`
`2504`	`2506`	`/* returned datum passed datum, no need to reparent */`

Comments

(0)