diff options
author | Victor Stinner <vstinner@python.org> | 2020-05-12 16:46:20 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-05-12 16:46:20 (GMT) |
commit | f453221c8b80e0570066a9375337f208d50e6406 (patch) | |
tree | 9ea682c978b6a302388c4df1b8174d5188e0b2d2 /Python | |
parent | 4c9ea093cd752a6687864674d34250653653f743 (diff) | |
download | cpython-f453221c8b80e0570066a9375337f208d50e6406.zip cpython-f453221c8b80e0570066a9375337f208d50e6406.tar.gz cpython-f453221c8b80e0570066a9375337f208d50e6406.tar.bz2 |
bpo-40602: Add _Py_HashPointerRaw() function (GH-20056)
Add a new _Py_HashPointerRaw() function which avoids replacing -1
with -2 to micro-optimize hash table using pointer keys: using
_Py_hashtable_hash_ptr() hash function.
Diffstat (limited to 'Python')
-rw-r--r-- | Python/hashtable.c | 2 | ||||
-rw-r--r-- | Python/pyhash.c | 14 |
2 files changed, 11 insertions, 5 deletions
diff --git a/Python/hashtable.c b/Python/hashtable.c index 1548c2e..90fe34e 100644 --- a/Python/hashtable.c +++ b/Python/hashtable.c @@ -109,7 +109,7 @@ _Py_hashtable_hash_ptr(struct _Py_hashtable_t *ht, const void *pkey) { void *key; _Py_HASHTABLE_READ_KEY(ht, pkey, key); - return (Py_uhash_t)_Py_HashPointer(key); + return (Py_uhash_t)_Py_HashPointerRaw(key); } diff --git a/Python/pyhash.c b/Python/pyhash.c index a6f42e7..3843079 100644 --- a/Python/pyhash.c +++ b/Python/pyhash.c @@ -129,16 +129,22 @@ _Py_HashDouble(double v) } Py_hash_t -_Py_HashPointer(const void *p) +_Py_HashPointerRaw(const void *p) { - Py_hash_t x; size_t y = (size_t)p; /* bottom 3 or 4 bits are likely to be 0; rotate y by 4 to avoid excessive hash collisions for dicts and sets */ y = (y >> 4) | (y << (8 * SIZEOF_VOID_P - 4)); - x = (Py_hash_t)y; - if (x == -1) + return (Py_hash_t)y; +} + +Py_hash_t +_Py_HashPointer(const void *p) +{ + Py_hash_t x = _Py_HashPointerRaw(p); + if (x == -1) { x = -2; + } return x; } |