]> granicus.if.org Git - postgresql/commit
Add valgrind suppressions for wcsrtombs optimizations
authorTomas Vondra <tomas.vondra@postgresql.org>
Sat, 17 Nov 2018 22:50:21 +0000 (23:50 +0100)
committerTomas Vondra <tomas.vondra@postgresql.org>
Sat, 17 Nov 2018 23:09:48 +0000 (00:09 +0100)
commitf57b94d9d0bbf42a18690d820f884dc0fc6bf79e
tree7f7dcae12a7115aff0664e21ff79edce5c6662ca
parent44dc82690d4e4ac92faed036c2c2867dbb6a88b3
Add valgrind suppressions for wcsrtombs optimizations

wcsrtombs (called through wchar2char from common functions like lower,
upper, etc.) uses various optimizations that may look like access to
uninitialized data, triggering valgrind reports.

For example AVX2 instructions load data in 256-bit chunks, and  gconv
does something similar with 32-bit chunks.  This is faster than accessing
the bytes one by one, and the uninitialized part of the buffer is not
actually used. So suppress the bogus reports.

The exact stack depends on possible optimizations - it might be AVX, SSE
(as in the report by Aleksander Alekseev) or something else. Hence the
last frame is wildcarded, to deal with this.

Backpatch all the way back to 9.4.

Author: Tomas Vondra
Discussion: https://www.postgresql.org/message-id/flat/90ac0452-e907-e7a4-b3c8-15bd33780e62%402ndquadrant.com
Discussion: https://www.postgresql.org/message-id/20180220150838.GD18315@e733.localdomain
src/tools/valgrind.supp