8000 GH-122548: Implement branch taken and not taken events for sys.monitoring by markshannon · Pull Request #122564 · python/cpython · GitHub
[go: up one dir, main page]
Skip to content

GH-122548: Implement branch taken and not taken events for sys.monitoring #122564

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 13 commits into from
Dec 19, 2024
Merged

GH-122548: Implement branch taken and not taken events for sys.monitoring #122564

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
c5a4b17
Add BRANCH_LEFT and BRANCH_RIGHT events to sys.monitoring for coverag…
markshannon Dec 11, 2024
2814c23
Fix reference counting in _PyMonitoring_RegisterCallback
markshannon Dec 13, 2024
9318ce1
Add new classes to globals check whitelist
markshannon Dec 13, 2024
1b8bb3c
Document new events
markshannon Dec 13, 2024
53ddd91
Fix up docs
markshannon Dec 13, 2024
bdfabc8
More docs updates
markshannon Dec 13, 2024
9cf3619
Fix typo
markshannon Dec 13, 2024
641f290
Fix alignment
markshannon Dec 13, 2024
f21effd
Update C-API docs
markshannon Dec 13, 2024
a604a08
Merge branch 'main' into monitoring-branch-taken
markshannon Dec 14, 2024
0278706
Add NOT_TAKEN in ADDOP_JUMP to tidy up a bit and make the code less e…
markshannon Dec 19, 2024
0258694
Tidy up test for conditional opcodes
markshannon Dec 19, 2024
7110ceb
Merge branch 'main' into monitoring-branch-taken
markshannon Dec 19, 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
Tidy up test for conditional opcodes
  • Loading branch information
@markshannon
markshannon committed Dec 19, 2024
commit 0258694d2509ddb87e5fb94946cfc7790a880faa
6 changes: 6 additions & 0 deletions Include/internal/pycore_opcode_utils.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,12 @@ extern "C" {
(opcode) == JUMP_BACKWARD || \
(opcode) == JUMP_BACKWARD_NO_INTERRUPT)

#define IS_CONDITIONAL_JUMP_OPCODE(opcode) \
((opcode) == POP_JUMP_IF_FALSE || \
(opcode) == POP_JUMP_IF_TRUE || \
(opcode) == POP_JUMP_IF_NONE || \
(opcode) == POP_JUMP_IF_NOT_NONE)

#define IS_SCOPE_EXIT_OPCODE(opcode) \
((opcode) == RETURN_VALUE || \
(opcode) == RAISE_VARARGS || \
Expand Down
9 changes: 2 additions & 7 deletions Python/codegen.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -409,13 +409,8 @@ codegen_addop_j(instr_sequence *seq, location loc,
if (_PyInstructionSequence_Addop(seq, opcode, target.id, loc) != SUCCESS) {
return ERROR;
}
switch (opcode) {
case POP_JUMP_IF_FALSE:
case POP_JUMP_IF_TRUE:
case POP_JUMP_IF_NONE:
case POP_JUMP_IF_NOT_NONE:
case FOR_ITER:
return _PyInstructionSequence_Addop(seq, NOT_TAKEN, 0, NO_LOCATION);
if (IS_CONDITIONAL_JUMP_OPCODE(opcode) || opcode == FOR_ITER) {
return _PyInstructionSequence_Addop(seq, NOT_TAKEN, 0, NO_LOCATION);
}
return SUCCESS;
}
Expand Down
0