8000 gh-93678: add _testinternalcapi.optimize_cfg() and test utils for compiler optimization unit tests by iritkatriel · Pull Request #96007 · python/cpython · GitHub
[go: up one dir, main page]
Skip to content

gh-93678: add _testinternalcapi.optimize_cfg() and test utils for compiler optimization unit tests #96007

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 11 commits into from
Aug 24, 2022
Merged

gh-93678: add _testinternalcapi.optimize_cfg() and test utils for compiler optimization unit tests #96007

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
7d44721
gh-93678: added _testinternalcapi.optimize_cfg() function and test ut…
iritkatriel Jul 1, 2022
e5cc736
📜🤖 Added by blurb_it.
blurb-it[bot] Aug 15, 2022
3089eed
Merge remote-tracking branch 'upstream/main' into optimize_func_1
iritkatriel Aug 15, 2022
12c1f57
regen
iritkatriel Aug 15, 2022
a68483a
Merge remote-tracking branch 'upstream/main' into optimize_func_1
iritkatriel Aug 22, 2022
2818fd3
long long --> uintptr_t, and added a comment clarifying that the poin…
iritkatriel Aug 23, 2022
4339460
Revert "long long --> uintptr_t, and added a comment clarifying that …
iritkatriel Aug 23, 2022
b45b11e
Revert "Revert "long long --> uintptr_t, and added a comment clarifyi…
iritkatriel Aug 23, 2022
fb746ce
one more cast
iritkatriel Aug 23, 2022
78f3fa9
fix whitespace
iritkatriel Aug 23, 2022
ebb10bb
calculate block ID and use it for label, instead of address
iritkatriel Aug 23, 2022
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
calculate block ID and use it for label, instead of address
  • Loading branch information
@iritkatriel
iritkatriel committed Aug 23, 2022
commit ebb10bb482c3c94c2f9f6f2a308f48f4abd17e22
15 changes: 7 additions & 8 deletions Python/compile.c
View file
8000 Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9691,8 +9691,12 @@ cfg_to_instructions(cfg_builder *g)
if (instructions == NULL) {
return NULL;
}
int lbl = 1;
for (basicblock *b = g->g_entryblock; b != NULL; b = b->b_next) {
PyObject *lbl = PyLong_FromUnsignedLongLong((uintptr_t)b);
b->b_label = lbl++;
}
for (basicblock *b = g->g_entryblock; b != NULL; b = b->b_next) {
PyObject *lbl = PyLong_FromLong(b->b_label);
if (lbl == NULL) {
goto error;
}
Expand All @@ -9704,14 +9708,9 @@ cfg_to_instructions(cfg_builder *g)
for (int i = 0; i < b->b_iused; i++) {
struct instr *instr = &b->b_instr[i];
struct location loc = instr->i_loc;
uintptr_t arg = instr->i_oparg;
if (HAS_TARGET(instr->i_opcode)) {
/* Use the address of the block as its unique ID (for the label) */
arg = (uintptr_t)instr->i_target;
}

int arg = HAS_TARGET(instr->i_opcode) ? instr->i_target->b_label : instr->i_oparg;
PyObject *inst_tuple = Py_BuildValue(
"(iLiiii)", instr->i_opcode, arg,
"(iiiiii)", instr->i_opcode, arg,
loc.lineno, loc.end_lineno,
loc.col_offset, loc.end_col_offset);
if (inst_tuple == NULL) {
Expand Down
0