Skip to content

gh-119726: JIT: re-use trampolines on AArch64#120250

Merged
brandtbucher merged 7 commits intopython:mainfrom
diegorusso:main
Jun 19, 2024
Merged

gh-119726: JIT: re-use trampolines on AArch64#120250
brandtbucher merged 7 commits intopython:mainfrom
diegorusso:main

Conversation

@diegorusso
Copy link
Contributor

@diegorusso diegorusso commented Jun 7, 2024 •
edited
Loading

When emitting AArch64 trampolines at the end of every data stencil, re-use existent ones fot the same symbol.

@diegorusso
pythongh-119726: JIT: re-use trampolines on AArch64
7942eab 
When emitting AArch64 trampolines at the end of every data stencil,
re-use existent ones fot the same symbol.
Fix the disassebly to reflect the "bl" instruction without the
relocation.
@diegorusso diegorusso requested a review from brandtbucher as a code owner June 7, 2024 22:20
@bedevere-app bedevere-app bot mentioned this pull request Jun 7, 2024
Closed
5 tasks
brandtbucher
brandtbucher reviewed Jun 8, 2024
View reviewed changes
Copy link
Member

@brandtbucher brandtbucher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work! Just a few thoughts I had while looking it over:

@brandtbucher brandtbucher self-assigned this Jun 8, 2024
@brandtbucher brandtbucher added performance Performance or resource usage interpreter-core (Objects, Python, Grammar, and Parser dirs) topic-JIT labels Jun 8, 2024
@brandtbucher
Copy link
Member

brandtbucher commented Jun 8, 2024

Pretty sure the CI failures are preexisting (we should probably confirm that those tests pass on native hardware, and add them to the ones skipped under emulation in another PR).

@diegorusso
Copy link
Contributor Author

diegorusso commented Jun 10, 2024 •
edited
Loading

Pretty sure the CI failures are preexisting (we should probably confirm that those tests pass on native hardware, and add them to the ones skipped under emulation in another PR).

We've tested it on our internal CI and we didn't see any problems. We tested it against Linux AArch64/x86 and MacOS arm64.

@brandtbucher
Copy link
Member

brandtbucher commented Jun 11, 2024

We've tested it on our internal CI and we didn't see any problems. We tested it against Linux AArch64/x86 and MacOS arm64.

Awesome. Want to open a PR to update Tools/jit/ignore-tests-emulated-linux.txt with the failing tests?

@diegorusso
Copy link
Contributor Author

diegorusso commented Jun 11, 2024

We've tested it on our internal CI and we didn't see any problems. We tested it against Linux AArch64/x86 and MacOS arm64.

Awesome. Want to open a PR to update Tools/jit/ignore-tests-emulated-linux.txt with the failing tests?

you have it: #120375

brandtbucher
brandtbucher reviewed Jun 11, 2024
View reviewed changes
Copy link
Member

@brandtbucher brandtbucher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Just two more suggestions:

@brandtbucher brandtbucher merged commit a0dce37 into python:main Jun 19, 2024
mrahtz pushed a commit to mrahtz/cpython that referenced this pull request Jun 30, 2024
@diegorusso @mrahtz
pythonGH-119726: Deduplicate JIT trampolines for out-of-range jumps (p…
c4193b2 
…ythonGH-120250)
noahbkim pushed a commit to hudson-trading/cpython that referenced this pull request Jul 11, 2024
@diegorusso @noahbkim
pythonGH-119726: Deduplicate JIT trampolines for out-of-range jumps (p…
6cc7eb9 
…ythonGH-120250)
estyxx pushed a commit to estyxx/cpython that referenced this pull request Jul 17, 2024
@diegorusso @estyxx
pythonGH-119726: Deduplicate JIT trampolines for out-of-range jumps (p…
47d8c29 
…ythonGH-120250)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@brandtbucher brandtbucher brandtbucher approved these changes

Assignees

@brandtbucher brandtbucher

Labels

interpreter-core (Objects, Python, Grammar, and Parser dirs) performance Performance or resource usage topic-JIT

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@diegorusso @brandtbucher