Skip to content

[WIP] bpo-17611: Move unwinding of stack from interpreter to compiler #2827

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

Closed
pitrou wants to merge 20 commits into from
Closed

[WIP] bpo-17611: Move unwinding of stack from interpreter to compiler #2827

Changes from 1 commit
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
656e4f3
bpo-17611: Move unwinding of stack from interpreter to compiler
pitrou Jul 23, 2017
c8068ab
Cleanup
pitrou Jul 23, 2017
b699364
Improve stack size calculation for SETUP_{EXCEPT,FINALLY}
pitrou Jul 23, 2017
541e7ce
Update frozen importlib
pitrou Jul 23, 2017
d3dc002
Remove useless code
pitrou Jul 23, 2017
b6d597c
Don't predict FOR_ITER as it breaks tracing without computed gotos
pitrou Jul 23, 2017
31e46dc
Remove dead code and fix tracing tests
pitrou Jul 23, 2017
693b939
Fix test_dis
pitrou Jul 23, 2017
d6ca2b9
Add JUMP_FINALLY to avoid finally block duplication
pitrou Jul 24, 2017
aefdbd1
Remove finally block duplication
pitrou Jul 24, 2017
9d12612
More precise computation of code object stack size
pitrou Jul 24, 2017
4ce87b6
JUMP_FINALLY pushes 6 values to be consistent with the effect of an e…
pitrou Jul 24, 2017
029f825
Exact stack size computation by popping stale exception state in RERAISE
pitrou Jul 24, 2017
bd0167f
Remove the now pointless POP_MANY
pitrou Jul 24, 2017
e5c1db6
Add comment for RERAISE
pitrou Jul 24, 2017
0ec9d12
Remove JUMP_FINALLY
pitrou Jul 24, 2017
9c3cfd0
Remove last block duplication
pitrou Jul 24, 2017
9a420d1
Get rid of END_ITER
pitrou Jul 24, 2017
336808c
Fix comments in frameobject.c
pitrou Jul 24, 2017
61ee3fa
Add stack size tests
pitrou Jul 29, 2017
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
Remove last block duplication
  • Loading branch information
@pitrou
pitrou committed Jul 24, 2017
commit 9c3cfd09ef8639e84faf34d48c0518a2e30c3448
16 changes: 4 additions & 12 deletions Python/compile.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2670,15 +2670,12 @@ compiler_continue(struct compiler *c)


/* Code generated for "try: <body> finally: <finalbody>" is as follows:
* XXX update this

SETUP_FINALLY F2
SETUP_FINALLY F
<code for body>
POP_BLOCK
F1:
<code for finalbody>
JUMP_ABSOLUTE EXIT
F2:
PUSH_NO_EXCEPT
F:
<code for finalbody>
RERAISE
EXIT:
Expand Down Expand Up @@ -2849,14 +2846,9 @@ compiler_try_except(struct compiler *c, stmt_ty s)
VISIT_SEQ(c, stmt, handler->v.ExceptHandler.body);
ADDOP(c, POP_BLOCK);
ADDOP(c, POP_EXCEPT);
ADDOP(c, PUSH_NO_EXCEPT);
compiler_pop_fblock(c, HANDLER_CLEANUP, cleanup_body);

/* name = None; del name */
ADDOP_O(c, LOAD_CONST, Py_None, consts);
compiler_nameop(c, handler->v.ExceptHandler.name, Store);
compiler_nameop(c, handler->v.ExceptHandler.name, Del);
ADDOP_JREL(c, JUMP_FORWARD, end);

/* finally: */
compiler_use_next_block(c, cleanup_end);
if (!compiler_push_finally_end(c, cleanup_end))
Expand Down