Pablo Galindo Salgado <pablog...@gmail.com> added the comment:

Thanks a lot, Brandt for doing the benchmarking!

I think that unpack_sequence is **very** affected by code locality. I was 
debugging it recently with Linux perf 
(https://perf.wiki.kernel.org/index.php/Main_Page) and it shows that is 
extremely affected by L1-L2 cache performance. I still need to understand why, 
but yeah, I am fine ignoring it.

>All things considered, I believe we should fix this... I personally view 
>memory leaks as second only to crashes and incorrect behavior in terms of 
>severity.

Yeah, I very much agree with you. Could you convert your patch into a PR?

----------

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

Reply via email to