Message186455
"Including the wmemcmp patch did not improve the times on MSC v.1600 32 bit - if anything, the performance was a little slower for the test I used:"
I tested my patch on Windows before the commit and I saw similar performances with and without wmemcmp().
I checked again and you are true: performances are *a little bit* worse using wmemcmp().
"Looking at the assembler, there is a real call to wmemcmp which adds some time and wmemcmp does not seem to be optimized compared to a simple loop."
You should be true. I reverted the patch for 16-bit wchar_t to use a dummy loop instead. 16-bit wchar_t can only be found on Windows, isn't it? |
|
Date |
User |
Action |
Args |
2013-04-09 21:57:09 | vstinner | set | recipients:
+ vstinner, georg.brandl, pitrou, ezio.melotti, ethan.furman, python-dev, serhiy.storchaka, Neil.Hodgson |
2013-04-09 21:57:09 | vstinner | set | messageid: <1365544629.84.0.46547354358.issue17615@psf.upfronthosting.co.za> |
2013-04-09 21:57:09 | vstinner | link | issue17615 messages |
2013-04-09 21:57:09 | vstinner | create | |
|