gh-131798: Split FOR_ITER_GEN into smaller uops#148372
gh-131798: Split FOR_ITER_GEN into smaller uops#148372Sacul0457 wants to merge 3 commits intopython:mainfrom
FOR_ITER_GEN into smaller uops#148372Conversation
Uh oh. you just caught a bug in the JIT optimizer. Would you like to fix it? The problem is that this line is wrong https://github.com/python/cpython/blob/main/Python/optimizer_symbols.c#L807 it should not be returning PyGen_Type, but rather NULL. Instead, _Py_uop_sym_get_probable_type should be the one returning PyGen_Type. If you want to fix it, just move the lines around amend this small test to |
|
Sure! |
|
Please open an issue and fix it in that PR. Thank you. |
I'm not sure how the optimizer knows the symbolic type on the first guard but it seems to work?