Skip to content

Oparg resume depth #7515

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
youknowone merged 8 commits into from
Mar 27, 2026
Merged

Oparg resume depth #7515

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
e462eea
Base resume context
ShaharNaveh Mar 27, 2026
009370b
Fixes for api change
ShaharNaveh Mar 27, 2026
e4af47d
Align codegen
ShaharNaveh Mar 27, 2026
a37bd3c
Align `frame.rs` to the api changes
ShaharNaveh Mar 27, 2026
d682618
fix jit
ShaharNaveh Mar 27, 2026
b5d3031
Use new oparg
ShaharNaveh Mar 27, 2026
fb7c109
Fix doc
ShaharNaveh Mar 27, 2026
07d019a
let `ir` to decide exception depth
ShaharNaveh Mar 27, 2026
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
Fixes for api change
  • Loading branch information
@ShaharNaveh
ShaharNaveh committed Mar 27, 2026
commit 009370b24236a8035f07a9a3b0436a66bfe45859
4 changes: 2 additions & 2 deletions crates/codegen/src/ir.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -363,7 +363,7 @@ impl CodeInfo {
}
}

let mut block_to_offset = vec![Label::new(0); blocks.len()];
let mut block_to_offset = vec![Label::from_u32(0); blocks.len()];
// block_to_index: maps block idx to instruction index (for exception table)
// This is the index into the final instructions array, including EXTENDED_ARG and CACHE
let mut block_to_index = vec![0u32; blocks.len()];
Expand All @@ -372,7 +372,7 @@ impl CodeInfo {
loop {
let mut num_instructions = 0;
for (idx, block) in iter_blocks(&blocks) {
block_to_offset[idx.idx()] = Label::new(num_instructions as u32);
block_to_offset[idx.idx()] = Label::from_u32(num_instructions as u32);
// block_to_index uses the same value as block_to_offset but as u32
// because lasti in frame.rs is the index into instructions array
// and instructions array index == byte offset (each instruction is 1 CodeUnit)
Expand Down