]> git.ipfire.org Git - thirdparty/postgresql.git/commit
llvmjit: Also copy parameter / return value attributes from template functions.
authorAndres Freund <andres@anarazel.de>
Thu, 15 Oct 2020 20:39:41 +0000 (13:39 -0700)
committerAndres Freund <andres@anarazel.de>
Thu, 15 Oct 2020 22:06:16 +0000 (15:06 -0700)
commitf3dee5b9aba6bd8cf3d5a13296495df7b273835c
treee6b32f219f6cd6d5b6aa3998ad0eb8fbd4bba368
parent3c4b5206850263ec0585db60bf5d17187b0dd513
llvmjit: Also copy parameter / return value attributes from template functions.

Previously we only copied the function attributes. That caused problems at
least on s390x: Because we didn't copy the 'zeroext' attribute for
ExecAggTransReparent()'s *IsNull parameters, expressions invoking it didn't
ensure that the upper bytes of the registers were zeroed. In the - relatively
rare - cases where not, ExecAggTransReparent() wrongly ended up in the
newValueIsNull branch due to the register not being zero. Subsequently causing
a crash.

It's quite possible that this would cause problems on other platforms, and in
other places than just ExecAggTransReparent() on s390x.

Thanks to Christoph (and the Debian project) for providing me with access to a
s390x machine, allowing me to debug this.

Reported-By: Christoph Berg
Author: Andres Freund
Discussion: https://postgr.es/m/20201015083246.kie5726xerdt3ael@alap3.anarazel.de
Backpatch: 11-, where JIT was added
src/backend/jit/llvm/llvmjit.c