Message266290
There's some difficulty in changing next_instr to an unsigned short* unless pointer arithmetic converts it back to unsigned char*. Code relies on f_lasti to be a byte index more than it relies on f_lasti to be -1 at first. Should this change include converting jump offsets to word increments? This patch seeks to be a minimal change to achieve the desired aligned/batched memory read behavior of NEXTOP/NEXTARG. Perhaps a clear description of how deep the changes should be is in order
It's also a bit hard to assert pointer alignment without failling back to stdint's uintptr_t. It may be sufficient to assert f_lasti is an even number |
|
Date |
User |
Action |
Args |
2016-05-24 23:44:15 | Demur Rumed | set | recipients:
+ Demur Rumed, mark.dickinson, vstinner, serhiy.storchaka, Adrian Wielgosik |
2016-05-24 23:44:15 | Demur Rumed | set | messageid: <1464133455.46.0.788626211033.issue27097@psf.upfronthosting.co.za> |
2016-05-24 23:44:15 | Demur Rumed | link | issue27097 messages |
2016-05-24 23:44:15 | Demur Rumed | create | |
|