JIT arm64 relocations does not handles large addresses
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 77.2k
- Forks
- 35.9k
- PR merge metrics
- PR metrics pending
Description
In https://github.com/python/cpython/pull/146906, the JIT inlines the address of the function pointer in operand0 however the arm64 relocation does not properly handles large addresses which needs a trampoline.
I think the fix is to generate trampoline for such cases if the address is too large.
Claude suggested the following fix which seems to work: https://gist.github.com/kumaraditya303/45ca837aa1367e65a3bfadca9fae039e
cc @markshannon
Linked PRs
- gh-147941
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Review the JIT arm64 relocation changes from PR 146906 and compare them with the proposed fix in the linked gist. Check linked PR 147941 for the current work; done means large function-pointer addresses are relocated correctly, including cases requiring a trampoline.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- compilers
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 25/100