Inada Naoki <[email protected]> added the comment:
When removed shash:
```
## small key
$ ./python -m pyperf timeit --compare-to ../cpython/python -s 'd={b"foo":1,
b"bar":2, b"buzz":3}' -- 'b"key" in d'
/home/inada-n/work/python/cpython/python: ..................... 23.2 ns +- 1.7
ns
/home/inada-n/work/python/remove-bytes-hash/python: ..................... 40.0
ns +- 1.5 ns
Mean +- std dev: [/home/inada-n/work/python/cpython/python] 23.2 ns +- 1.7 ns
-> [/home/inada-n/work/python/remove-bytes-hash/python] 40.0 ns +- 1.5 ns:
1.73x slower
## large key
$ ./python -m pyperf timeit --compare-to ../cpython/python -s 'd={b"foo":1,
b"bar":2, b"buzz":3};k=b"key"*100' -- 'k in d'
/home/inada-n/work/python/cpython/python: ..................... 22.3 ns +- 1.2
ns
/home/inada-n/work/python/remove-bytes-hash/python: ..................... 108
ns +- 2 ns
Mean +- std dev: [/home/inada-n/work/python/cpython/python] 22.3 ns +- 1.2 ns
-> [/home/inada-n/work/python/remove-bytes-hash/python] 108 ns +- 2 ns: 4.84x
slower
```
I will reconsider the removal before remove the cache.
We changed code object too often. If Python 3.13 don't use so much bytes
objects, we don't need to remove the hash to save some RAM.
----------
Added file: https://bugs.python.org/file50649/remove-bytes-hash.patch
_______________________________________
Python tracker <[email protected]>
<https://bugs.python.org/issue46864>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe:
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
