Skip to content

GH-136410: Faster side exits #136411

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
markshannon merged 15 commits into from
Aug 1, 2025
Merged

GH-136410: Faster side exits #136411

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
a281775
Faster side exits.
markshannon Jul 7, 2025
ec7f0e2
Fix copy-and-paste error
markshannon Jul 7, 2025
911520a
Check validity at start of trace
markshannon Jul 7, 2025
73832b2
Tidy up
markshannon Jul 8, 2025
a9297a0
Null test before use
markshannon Jul 8, 2025
47b53d1
Update asserts
markshannon Jul 8, 2025
2564d41
Properly fix assert
markshannon Jul 8, 2025
779bd7d
Don't make object immortal until it is fully initialized
markshannon Jul 8, 2025
db336ce
Merge branch 'main' into fast-side-exits
markshannon Jul 21, 2025
0b05acf
Free cold executor when closing of interpreter
markshannon Jul 21, 2025
142bab6
Add NULL check
markshannon Jul 21, 2025
af696ea
Free cold executor after final GC
markshannon Jul 22, 2025
264559a
Add empty _PyExecutor_Free for non-jit builds
markshannon Jul 22, 2025
989a882
Clear exit if START_EXECUTOR detects that the executor is invalid
markshannon Jul 30, 2025
656ada0
Add some asserts
markshannon Jul 30, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update asserts
  • Loading branch information
@markshannon
markshannon committed Jul 8, 2025
commit 47b53d1063036123f3cf072eaeb3f29c9e1086ca
2 changes: 1 addition & 1 deletion Python/jit.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -553,7 +553,7 @@ _PyJIT_Compile(_PyExecutorObject *executor, const _PyUOpInstruction trace[], siz
group->emit(code, data, executor, NULL, &state);
code += group->code_size;
data += group->data_size;
assert(trace[0].opcode == _START_EXECUTOR);
assert(trace[0].opcode == _START_EXECUTOR || trace[0].opcode == _COLD_EXIT);
for (size_t i = 0; i < length; i++) {
const _PyUOpInstruction *instruction = &trace[i];
group = &stencil_groups[instruction->opcode];
Expand Down
2 changes: 1 addition & 1 deletion Python/optimizer.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1130,7 +1130,7 @@ sanity_check(_PyExecutorObject *executor)
}
bool ended = false;
uint32_t i = 0;
CHECK(executor->trace[0].opcode == _START_EXECUTOR);
CHECK(executor->trace[0].opcode == _START_EXECUTOR || trace[0].opcode == _COLD_EXIT);
for (; i < executor->code_size; i++) {
const _PyUOpInstruction *inst = &executor->trace[i];
uint16_t opcode = inst->opcode;
Expand Down
Loading