Skip to content

Commit

Permalink
MDEV-9872: Valgrind supports CRC32B and CRC32Q since valgrind-3.6.1
Browse files Browse the repository at this point in the history
We don't need to drop down to unoptimized crc because of valgrind now.

Valgrind-3.6.1 was released 16 February 2011.

The Power8 ASM instructions seem to be supported in 3.9.0 (31 October 2013).

Signed-off-by: Daniel Black <daniel.black@au.ibm.com>
  • Loading branch information
grooverdan committed Dec 4, 2016
1 parent 6d12569 commit 410bf82
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion storage/innobase/ut/ut0crc32.cc
Original file line number Diff line number Diff line change
Expand Up @@ -705,7 +705,7 @@ ut_crc32_init()
ut_crc32_byte_by_byte = ut_crc32_byte_by_byte_sw;
ut_crc32_implementation = "Using generic crc32 instructions";

#if defined(__GNUC__) && defined(__x86_64__) && !defined(UNIV_DEBUG_VALGRIND)
#if defined(__GNUC__) && defined(__x86_64__)
uint32_t vend[3];
uint32_t model;
uint32_t family;
Expand Down
2 changes: 1 addition & 1 deletion storage/xtradb/ut/ut0crc32.cc
Original file line number Diff line number Diff line change
Expand Up @@ -303,7 +303,7 @@ ut_crc32_init()
ut_crc32 = ut_crc32_slice8;
ut_crc32_implementation = "Using generic crc32 instructions";

#if defined(__GNUC__) && defined(__x86_64__) && !defined(UNIV_DEBUG_VALGRIND)
#if defined(__GNUC__) && defined(__x86_64__)
ib_uint32_t vend[3];
ib_uint32_t model;
ib_uint32_t family;
Expand Down

0 comments on commit 410bf82

Please sign in to comment.