Inada Naoki <songofaca...@gmail.com> added the comment:

I had suspected that pypeformance just don't have enough workload for non-small 
int.

For example, spectral_norm is integer heavy + some float warkload. But 
bm_spectral_norm uses `DEFAULT_N = 130`. So most integers are fit into smallint 
cache.

On the othar hand, spectral_norm in the benchmarkgame uses N=5500.
https://benchmarksgame-team.pages.debian.net/benchmarksgame/program/spectralnorm-python3-8.html

So I ran the benchmark on my machine:

master:
real    1m24.647s
user    5m37.515s

patched:
real    1m19.033s
user    5m14.682s

master+increased small int from [-5, 256] to [-9, 1024]
real    1m23.742s
user    5m33.569s


314.682/337.515 = 0.9323496733478512. So ther is only 7% speedup even when 
N=5500.

After all, I think it is doubtful. Let's stop this idea until situation is  
changed.

----------

_______________________________________
Python tracker <rep...@bugs.python.org>
<https://bugs.python.org/issue24165>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to