diff options
| author | Tomas Vondra <tomas.vondra@postgresql.org> | 2019-01-19 20:49:51 +0100 |
|---|---|---|
| committer | Tomas Vondra <tomas.vondra@postgresql.org> | 2019-01-19 20:49:51 +0100 |
| commit | d6ef7fe75c1fb7ccc4ad8e3dd38d4e84d6fe9b9f (patch) | |
| tree | 0ea13cb69d893cfb87af53e3754acc67c4ba91b4 | |
| parent | 0301db623de076f9fb17189daaeb9202a05865bf (diff) | |
| download | postgresql-d6ef7fe75c1fb7ccc4ad8e3dd38d4e84d6fe9b9f.tar.gz postgresql-d6ef7fe75c1fb7ccc4ad8e3dd38d4e84d6fe9b9f.zip | |
Revert "Add valgrind suppressions for wcsrtombs optimizations"
This reverts commit d3bbc4b96a5b4d055cf636596c6865913a099929.
Per discussion, it's not desirable to add valgrind suppressions for
outside our own code base (e.g. glibc in this case), especially when
the suppressions may be platform-specific. There are better ways to
deal with that, e.g. by providing local suppressions.
Discussion: https://www.postgresql.org/message-id/flat/90ac0452-e907-e7a4-b3c8-15bd33780e62%402ndquadrant.com
| -rw-r--r-- | src/tools/valgrind.supp | 36 |
1 files changed, 0 insertions, 36 deletions
diff --git a/src/tools/valgrind.supp b/src/tools/valgrind.supp index 85122315393..ec47a228ae5 100644 --- a/src/tools/valgrind.supp +++ b/src/tools/valgrind.supp @@ -212,39 +212,3 @@ Memcheck:Cond fun:PyObject_Realloc } - -# wcsrtombs uses some clever optimizations internally, which to valgrind -# may look like access to uninitialized data. For example AVX2 instructions -# load data in 256-bit chunks, irrespectedly of wchar length. gconv does -# somethink similar by loading data in 32bit chunks and then shifting the -# data internally. Neither of those actually uses the uninitialized part -# of the buffer, as far as we know. -# -# https://www.postgresql.org/message-id/90ac0452-e907-e7a4-b3c8-15bd33780e62@2ndquadrant.com - -{ - wcsnlen_optimized - Memcheck:Cond - ... - fun:wcsrtombs - fun:wcstombs - fun:wchar2char -} - -{ - wcsnlen_optimized_addr32 - Memcheck:Addr32 - ... - fun:wcsrtombs - fun:wcstombs - fun:wchar2char -} - -{ - gconv_transform_internal - Memcheck:Cond - fun:__gconv_transform_internal_utf8 - fun:wcsrtombs - fun:wcstombs - fun:wchar2char -} |