Message262658
Demur, I think you're on the right track here. It will nice to be rid of the HAVE_ARGUMENT tests and to not have to decode the arguments one-byte at a time. Overall, the patch looks good (although it includes several small unrelated changes). Besides the speed benefit, the code looks cleaner than before.
I was surprised to see that the peephole optimizer grew larger, but the handling of extended arguments is likely worth it even though it adds several new wordy chunks of code.
When it comes to benchmarks, expect a certain amount of noise (especially from those that use I/O or that exercise the C-API more than the pure python bytecode). |
|
Date |
User |
Action |
Args |
2016-03-30 10:13:26 | rhettinger | set | recipients:
+ rhettinger, brett.cannon, georg.brandl, ncoghlan, vstinner, benjamin.peterson, serhiy.storchaka, yselivanov, abarnert, Demur Rumed |
2016-03-30 10:13:26 | rhettinger | set | messageid: <1459332806.63.0.951946814011.issue26647@psf.upfronthosting.co.za> |
2016-03-30 10:13:26 | rhettinger | link | issue26647 messages |
2016-03-30 10:13:26 | rhettinger | create | |
|