Skip to content

Bytecode enum named oparg #7294

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 26 commits into from
Mar 3, 2026
Merged

Bytecode enum named oparg #7294

Changes from 1 commit
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
59a8471
Align real opcodes
ShaharNaveh Mar 1, 2026
5c99d26
Align pseudoes
ShaharNaveh Mar 1, 2026
511bc2e
fix fmt_dis
ShaharNaveh Mar 1, 2026
6b9966b
Compiler compiles
ShaharNaveh Mar 1, 2026
ac64b1a
Fix some of ir
ShaharNaveh Mar 1, 2026
927daa1
Fix some of compile.rs
ShaharNaveh Mar 1, 2026
8845094
codegen compiles
ShaharNaveh Mar 1, 2026
bf1dcfa
VM compiles
ShaharNaveh Mar 2, 2026
6983124
Merge remote-tracking branch 'upstream/main' into bytecode-enum-named…
ShaharNaveh Mar 2, 2026
18b0dfa
Fix compiler-core
ShaharNaveh Mar 2, 2026
96dd420
fix merge
ShaharNaveh Mar 2, 2026
ede9094
fix more merge
ShaharNaveh Mar 2, 2026
0bf36db
fix stdlib
ShaharNaveh Mar 2, 2026
aad0560
Fix some jit
ShaharNaveh Mar 2, 2026
8af88fe
Fix more jit
ShaharNaveh Mar 2, 2026
358d466
Fix some jit tests
ShaharNaveh Mar 2, 2026
cf22b78
More jit fixes
ShaharNaveh Mar 2, 2026
64a3ef8
Make sure no instruction is emitted using `Foo(val)`
ShaharNaveh Mar 2, 2026
de0a702
Add comment
ShaharNaveh Mar 2, 2026
faf47b3
Merge remote-tracking branch 'upstream/main' into bytecode-enum-named…
ShaharNaveh Mar 2, 2026
ffe71f9
Fix typo
ShaharNaveh Mar 2, 2026
a828ab1
Add `consti` to known words
ShaharNaveh Mar 2, 2026
04b7c4a
Merge remote-tracking branch 'upstream/main' into bytecode-enum-named…
ShaharNaveh Mar 3, 2026
f9e5530
Merge remote-tracking branch 'upstream/main' into bytecode-enum-named…
ShaharNaveh Mar 3, 2026
7865718
Fix more
ShaharNaveh Mar 3, 2026
0ae8b08
Fix bad merge
ShaharNaveh Mar 3, 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
Fix compiler-core
  • Loading branch information
@ShaharNaveh
ShaharNaveh committed Mar 2, 2026
commit 18b0dfa266fdfbfe31094d318fd30138dca7b147
50 changes: 30 additions & 20 deletions crates/compiler-core/src/bytecode/instruction.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -577,7 +577,9 @@ impl Instruction {
| Self::LoadAttrNondescriptorWithValues
| Self::LoadAttrProperty
| Self::LoadAttrSlot
| Self::LoadAttrWithHint => Self::LoadAttr { idx: Arg::marker() },
| Self::LoadAttrWithHint => Self::LoadAttr {
namei: Arg::marker(),
},
// BINARY_OP specializations
Self::BinaryOpAddFloat
| Self::BinaryOpAddInt
Expand Down Expand Up @@ -615,11 +617,11 @@ impl Instruction {
| Self::CallStr1
| Self::CallTuple1
| Self::CallType1 => Self::Call {
nargs: Arg::marker(),
argc: Arg::marker(),
},
// CALL_KW specializations
Self::CallKwBoundMethod | Self::CallKwNonPy | Self::CallKwPy => Self::CallKw {
nargs: Arg::marker(),
argc: Arg::marker(),
},
// TO_BOOL specializations
Self::ToBoolAlwaysTrue
Expand All @@ -629,48 +631,56 @@ impl Instruction {
| Self::ToBoolNone
| Self::ToBoolStr => Self::ToBool,
// COMPARE_OP specializations
Self::CompareOpFloat | Self::CompareOpInt | Self::CompareOpStr => {
Self::CompareOp { op: Arg::marker() }
}
Self::CompareOpFloat | Self::CompareOpInt | Self::CompareOpStr => Self::CompareOp {
opname: Arg::marker(),
},
// CONTAINS_OP specializations
Self::ContainsOpDict | Self::ContainsOpSet => Self::ContainsOp(Arg::marker()),
Self::ContainsOpDict | Self::ContainsOpSet => Self::ContainsOp {
invert: Arg::marker(),
},
// FOR_ITER specializations
Self::ForIterGen | Self::ForIterList | Self::ForIterRange | Self::ForIterTuple => {
Self::ForIter {
target: Arg::marker(),
delta: Arg::marker(),
}
}
// LOAD_GLOBAL specializations
Self::LoadGlobalBuiltin | Self::LoadGlobalModule => Self::LoadGlobal(Arg::marker()),
Self::LoadGlobalBuiltin | Self::LoadGlobalModule => Self::LoadGlobal {
namei: Arg::marker(),
},
// STORE_ATTR specializations
Self::StoreAttrInstanceValue | Self::StoreAttrSlot | Self::StoreAttrWithHint => {
Self::StoreAttr { idx: Arg::marker() }
Self::StoreAttr {
namei: Arg::marker(),
}
}
// LOAD_SUPER_ATTR specializations
Self::LoadSuperAttrAttr | Self::LoadSuperAttrMethod => {
Self::LoadSuperAttr { arg: Arg::marker() }
}
Self::LoadSuperAttrAttr | Self::LoadSuperAttrMethod => Self::LoadSuperAttr {
namei: Arg::marker(),
},
// STORE_SUBSCR specializations
Self::StoreSubscrDict | Self::StoreSubscrListInt => Self::StoreSubscr,
// UNPACK_SEQUENCE specializations
Self::UnpackSequenceList | Self::UnpackSequenceTuple | Self::UnpackSequenceTwoTuple => {
Self::UnpackSequence {
size: Arg::marker(),
count: Arg::marker(),
}
}
// SEND specializations
Self::SendGen => Self::Send {
target: Arg::marker(),
delta: Arg::marker(),
},
// LOAD_CONST specializations
Self::LoadConstImmortal | Self::LoadConstMortal => {
Self::LoadConst { idx: Arg::marker() }
}
Self::LoadConstImmortal | Self::LoadConstMortal => Self::LoadConst {
consti: Arg::marker(),
},
// RESUME specializations
Self::ResumeCheck => Self::Resume { arg: Arg::marker() },
Self::ResumeCheck => Self::Resume {
context: Arg::marker(),
},
// JUMP_BACKWARD specializations
Self::JumpBackwardJit | Self::JumpBackwardNoJit => Self::JumpBackward {
target: Arg::marker(),
delta: Arg::marker(),
},
// Instrumented opcodes map back to their base
_ => match self.to_base() {
Expand Down