Skip to content

Align _opcode_metadata.py to 3.14.3 #7456

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 18, 2026
Merged

Align _opcode_metadata.py to 3.14.3 #7456

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
a7327b6
Align `_opcode_metadata.py` to 3.14.3
ShaharNaveh Mar 16, 2026
6393ac0
Unmark passing test
ShaharNaveh Mar 17, 2026
e59d6cd
Ensure python 3.14 runs on CI
ShaharNaveh Mar 17, 2026
ba3fde9
Merge remote-tracking branch 'upstream/main' into align-opcode-metadata
ShaharNaveh Mar 17, 2026
cf536b7
Update banner
ShaharNaveh Mar 17, 2026
5f8f38d
Fix `test__opcode.py`
ShaharNaveh Mar 17, 2026
4c9193c
Adjust generate script
ShaharNaveh Mar 17, 2026
2aef7fc
Fix docs
ShaharNaveh Mar 17, 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
Fix docs
  • Loading branch information
@ShaharNaveh
ShaharNaveh committed Mar 17, 2026
commit 2aef7fca1a65721eea5608a9883520780c5ba4bc
5 changes: 3 additions & 2 deletions crates/compiler-core/src/bytecode/instruction.rs
View file
Open in desktop
Copy link
Copy Markdown
Contributor Author

@ShaharNaveh ShaharNaveh Mar 17, 2026 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes to this file are purely "cosmetic". Altering the arms order (and variants order) so the generated output will match CPython

Original file line number Diff line number Diff line change
Expand Up @@ -555,7 +555,8 @@ impl Instruction {
_ => return None,
})
}
/// Map a specialized or instrumented opcode back to its adaptive (base) variant.

/// Map a specialized opcode back to its adaptive (base) variant.
/// `_PyOpcode_Deopt`
pub const fn deopt(self) -> Option<Self> {
Copy link
Copy Markdown
Contributor Author

@ShaharNaveh ShaharNaveh Mar 17, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have extracted the specializations depot part of the deoptimize method into a separate one called deopt.
The behavior of deoptimize did not change

Some(match self {
Expand Down Expand Up @@ -681,7 +682,7 @@ impl Instruction {
})
}

/// Map a specialized opcode back to its adaptive (base) variant.
/// Map a specialized or instrumented opcode back to its adaptive (base) variant.
pub const fn deoptimize(self) -> Self {
match self.deopt() {
Some(v) => v,
Expand Down
Loading