8000 GH-118093: Better handling of short and mid-loop traces by brandtbucher · Pull Request #122252 · python/cpython · GitHub
[go: up one dir, main page]
Skip to content

GH-118093: Better handling of short and mid-loop traces #122252

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
merged 10 commits into from
Jul 29, 2024
Merged

GH-118093: Better handling of short and mid-loop traces #122252

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
f8637ce
Allow one backward jump per trace, anywhere
brandtbucher Jul 17, 2024
f1f5d13
Use progress, rather than trace length, to reject traces
brandtbucher Jul 17, 2024
159e46b
Add missing RESERVEs
brandtbucher Jul 25, 2024
511ec09
Clean up overflow recovery
brandtbucher Jul 25, 2024
ba3d68b
Add asserts back and teach is_resume about ENTER_EXECUTOR
brandtbucher Jul 26, 2024
7c404a8
Guard _Py_GetExecutor on non-tier-two builds
brandtbucher Jul 26, 2024
1606abd
Revert "Guard _Py_GetExecutor on non-tier-two builds"
brandtbucher Jul 29, 2024
6901f60
Revert "Add asserts back and teach is_resume about ENTER_EXECUTOR"
brandtbucher Jul 29, 2024
9bcc643
Add comment to removed asserts
brandtbucher Jul 29, 2024
cd1f4d2
Catch up with main
brandtbucher Jul 29, 2024
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
Guard _Py_GetExecutor on non-tier-two builds
  • Loading branch information
@brandtbucher
brandtbucher committed Jul 26, 2024
commit 7c404a8dea42300d90f81602c09058898fc0bcf5
2 changes: 2 additions & 0 deletions Objects/genobject.c
View file
Open in desktop
4AC6
Original file line number Diff line number Diff line change
Expand Up @@ -331,15 +331,17 @@
is_resume(_PyInterpreterFrame *frame, uint8_t *oparg_p)
{
PyCodeObject *code = _PyFrame_GetCode(frame);
int offset = frame->instr_ptr - _PyCode_CODE(code);

Check warning on line 334 in Objects/genobject.c

View workflow job for this annotation

GitHub Actions / Windows / build (arm64) 

'initializing': conversion from '__int64' to 'int', possible loss of data [D:\a\cpython\cpython\PCbuild\pythoncore.vcxproj]

Check warning on line 334 in Objects/genobject.c

View workflow job for this annotation

GitHub Actions / Windows (free-threading) / build (arm64) 

'initializing': conversion from '__int64' to 'int', possible loss of data [D:\a\cpython\cpython\PCbuild\pythoncore.vcxproj]

Check warning on line 334 in Objects/genobject.c

View workflow job for this annotation

GitHub Actions / Windows / build and test (x64) 

'initializing': conversion from '__int64' to 'int', possible loss of data [D:\a\cpython\cpython\PCbuild\pythoncore.vcxproj]

Check warning on line 334 in Objects/genobject.c

View workflow job for this annotation

GitHub Actions / Windows (free-threading) / build and test (x64) 

'initializing': conversion from '__int64' to 'int', possible loss of data [D:\a\cpython\cpython\PCbuild\pythoncore.vcxproj]
uint8_t opcode = _Py_GetBaseOpcode(code, offset);
uint8_t oparg = frame->instr_ptr->op.arg;
#ifdef _Py_TIER2
if (opcode == ENTER_EXECUTOR) {
_PyExecutorObject *executor = _Py_GetExecutor(code, sizeof(_Py_CODEUNIT) * offset);
opcode = _PyOpcode_Deopt[executor->vm_data.opcode];
oparg = executor->vm_data.oparg;
Py_DECREF(executor);
}
#endif
if (opcode == RESUME) {
*oparg_p = oparg;
return true;
Expand All @@ -353,7 +355,7 @@
if (gen->gi_frame_state == FRAME_SUSPENDED_YIELD_FROM) {
_PyInterpreterFrame *frame = &gen->gi_iframe;
#ifndef NDEBUG
uint8_t oparg;

Check warning on line 358 in Objects/genobject.c

View workflow job for this annotation

GitHub Actions / Ubuntu SSL tests with OpenSSL (1.1.1w) 

‘oparg’ may be used uninitialized in this function [-Wmaybe-uninitialized]

Check warning on line 358 in Objects/genobject.c

View workflow job for this annotation

GitHub Actions / Ubuntu SSL tests with OpenSSL (3.0.13) 

‘oparg’ may be used uninitialized in this function [-Wmaybe-uninitialized]

Check warning on line 358 in Objects/genobject.c

View workflow job for this annotation

GitHub Actions / Hypothesis tests on Ubuntu 

‘oparg’ may be used uninitialized in this function [-Wmaybe-uninitialized]

Check warning on line 358 in Objects/genobject.c

View workflow job for this annotation

GitHub Actions / Ubuntu SSL tests with OpenSSL (3.1.5) 

‘oparg’ may be used uninitialized in this function [-Wmaybe-uninitialized]

Check warning on line 358 in Objects/genobject.c

View workflow job for this annotation

GitHub Actions / Ubuntu SSL tests with OpenSSL (3.2.1) 

‘oparg’ may be used uninitialized in this function [-Wmaybe-uninitialized]

Check warning on line 358 in Objects/genobject.c

View workflow job for this annotation

GitHub Actions / Ubuntu / build and test 

‘oparg’ may be used uninitialized in this function [-Wmaybe-uninitialized]

Check warning on line 358 in Objects/genobject.c

View workflow job for this annotation

GitHub Actions / Ubuntu (free-threading) / build and test 

‘oparg’ may be used uninitialized in this function [-Wmaybe-uninitialized]
assert(is_resume(frame, &oparg));
assert((oparg & RESUME_OPARG_LOCATION_MASK) >= RESUME_AFTER_YIELD_FROM);
#endif
Expand Down
Loading
0