]> git.ipfire.org Git - thirdparty/valgrind.git/commit
ppc stxsibx and stxsihx instructions write too much data
authorMark Wielaard <mark@klomp.org>
Tue, 15 Dec 2020 10:49:58 +0000 (11:49 +0100)
committerMark Wielaard <mark@klomp.org>
Tue, 15 Dec 2020 10:57:21 +0000 (11:57 +0100)
commitab257bc49a6c8beefa794470446f917ec441f718
tree29f87ad3bc1bf6087e551af3e72ac061ee504cfc
parentcb52fee5ddbc2c0e936fd1efe5107a1afcf375cf
ppc stxsibx and stxsihx instructions write too much data

stxsibx (Store VSX Scalar as Integer Byte Indexed X-form) is implemented
by first reading a whole word, merging in the new byte, and then writing
out the whole word. Causing memcheck to warn when the destination might
have room for less than 8 bytes.

The stxsihx (Store VSX Scalar as Integer Halfword Indexed X-form)
instruction does something similar reading and then writing a full
word instead of a half word.

The code can be simplified (and made more correct) by storing the byte
(or half-word) directly, IRStmt_Store seems fine to store byte or half
word sized data, and so seems the ppc backend.

https://bugs.kde.org/show_bug.cgi?id=430354
NEWS
VEX/priv/guest_ppc_toIR.c