| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
Rename Py_TAIL_CALL_INTERP to _Py_TAIL_CALL_INTERP.
|
@Fidget-Spinner: I blindly replaced Py_TAIL_CALL_INTERP text with _Py_TAIL_CALL_INTERP. I didn't test my change. |
Sorry, something went wrong.
|
Thanks. I'll approve once CI passes. |
Sorry, something went wrong.
|
I built Python with tail call on Linux using ./configure --with-tail-call-interp CC=clang LD=clang && make. I checked that ./python -m test test_sys -v still pass. |
Sorry, something went wrong.
Don't worry, we have CI for the tail calling interpreter on most platforms, so you can just let it run :) |
Sorry, something went wrong.
|
@hugovk: This change mostly/only makes sense if it's backported to 3.14. Would you accept this late C API change in 3.14? It makes the macro private. |
Sorry, something went wrong.
|
Maybe it's ok to leave the macro in 3.14 and remove it in 3.15. No one should rely on this macro :-) |
Sorry, something went wrong.
|
Thanks @vstinner for the PR 🌮🎉.. I'm working now to backport this PR to: 3.14. |
Sorry, something went wrong.
|
Sorry, @vstinner, I could not cleanly backport this to 3.14 due to a conflict. cherry_picker 6504f20ccedf0b27275327e72698dee5f0c75ba8 3.14 |
Sorry, something went wrong.
Rename Py_TAIL_CALL_INTERP to _Py_TAIL_CALL_INTERP. (cherry picked from commit 6504f20)
|
GH-139120 is a backport of this pull request to the 3.14 branch. |
Sorry, something went wrong.
|
@Fidget-Spinner suggested me to merge this change in the main branch, so I did it :-) |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Rename Py_TAIL_CALL_INTERP to _Py_TAIL_CALL_INTERP.