GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | |||||||||||||||||||||||||||||||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | |||||||||||||||||||||||||||||||||||
GeezerVison is showing github.com — gh-133171: Re-enable JUMP_BACKWARD to free-threading build by corona10 · Pull Request #137800 · python/cpython Conversation
|
|
@savannahostrowski @diegorusso @Fidget-Spinner @brandtbucher To the JIT team — I’m not sure who’s currently working on this, but it might be a good starting point to add JIT support. My next goal will be to make the tier-two instructions thread-safe. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
I think we should start by running the tests in CI for JIT+FT current build and see what passes/fails before doing this |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
We disabled FT + JIT build from #133179 do you think that we have to listing the current status? |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
This makes sense but actually the default build's JIT can't even run with the optimizer turned off! There's some segfaults when we turn off the optimizer lol. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
How can I trigger this? |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Run the test suite with |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Just for the record at default build: |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Ah never mind, it need to be transformed tier 2 code first. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
d855944 to
d470ce9
Compare
There was an error while loading. Please reload this page.
|
@Fidget-Spinner @brandtbucher I noticed that JIT x FT still doesn’t build even with the entire optimizer path disabled, so fix it temporarily. I’ve added a direct fallback to Tier 1; it’s slow, but fine for now. Let’s re-enable _PyOptimizer_Optimize step by step. To do that, we also need to re-enable CI (just for build testing) |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
cc @savannahostrowski Once this PR is merged, others can start collaborating on the full FT x JIT work separately. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
| static inline int | ||
| PyStackRef_RefcountOnObject(_PyStackRef ref) | ||
| { | ||
| return (ref.bits & Py_TAG_REFCNT) == 0; |
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
I think this implementation is fine, but I'm not 100% sure
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
If there are no major objections to this PR, I am going to merge this PR (maybe next week) |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
| } | ||
|
|
||
| static inline int | ||
| PyStackRef_RefcountOnObject(_PyStackRef ref) |
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Just personal interest, correct me if I'm wrong.
It seems we don;t call this function yet right? Is this prepared for a future hook?
This function seems be trigger at
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
It seems we don;t call this function yet right? Is this prepared for a future hook?
Yeah correct, but we can't build if this function is not added. That's all at this moment.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.