database/rocksdb
1
0
Fork 0
mirror of https://github.com/facebook/rocksdb.git synced 2026-10-23 08:09:02 +03:00
Code Activity

Force-inline xxph3 mix helpers on the NPHash64/Hash64 hot path (#14981)

Browse Source 
Summary: Pull Request resolved: https://github.com/facebook/rocksdb/pull/14981

Reviewed By: pdillinger

Differential Revision: D112965953

fbshipit-source-id: 9645d9287a9aba987bf1b37752df7f9dcb3a9669
This commit is contained in:
generatedunixname1395027625275998
2026-07-29 14:21:15 -07:00
committed by meta-codesync[bot]
parent 4b9070bccb
commit ba81144d8d
1 changed files with 8 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
util/xxph3.h
View File

@@ -943,8 +943,10 @@ XXPH_ALIGN(64) static const xxh_u8 kSecret[XXPH_SECRET_DEFAULT_SIZE] = {
*/
#if defined(__GNUC__) && !defined(__clang__) && defined(__i386__)
__attribute__((__target__("no-sse")))
#endif
static XXPH128_hash_t
#else
XXPH_FORCE_INLINE XXPH128_hash_t
#endif
XXPH_mult64to128(xxh_u64 lhs, xxh_u64 rhs)
{
/*
@@ -1052,13 +1054,14 @@ XXPH_mult64to128(xxh_u64 lhs, xxh_u64 rhs)
*
* Does a 64-bit to 128-bit multiply, then XOR folds it.
* The reason for the separate function is to prevent passing
* too many structs around by value. This will hopefully inline
* the multiply, but we don't force it.
* too many structs around by value.
*/
#if defined(__GNUC__) && !defined(__clang__) && defined(__i386__)
__attribute__((__target__("no-sse")))
#endif
static xxh_u64
#else
XXPH_FORCE_INLINE xxh_u64
#endif
XXPH3_mul128_fold64(xxh_u64 lhs, xxh_u64 rhs)
{
XXPH128_hash_t product = XXPH_mult64to128(lhs, rhs);
@@ -1066,7 +1069,7 @@ XXPH3_mul128_fold64(xxh_u64 lhs, xxh_u64 rhs)
}
static XXPH64_hash_t XXPH3_avalanche(xxh_u64 h64)
XXPH_FORCE_INLINE XXPH64_hash_t XXPH3_avalanche(xxh_u64 h64)
{
h64 ^= h64 >> 37;
h64 *= PRIME64_3;