1berry/ruby
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

YJIT: Fix clippy::redundant_locals

Browse Source 
> note: `#[deny(clippy::redundant_locals)]` on by default

On Rust 1.73.0.
This commit is contained in:
Alan Wu 2023-10-17 17:49:45 -04:00 committed by Alan Wu
parent 2abd061e8b
commit 36ee5d8ca8
1 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
yjit/src/backend/ir.rs
View File

@ -1056,11 +1056,10 @@ impl Assembler
/// Append an instruction onto the current list of instructions and update /// Append an instruction onto the current list of instructions and update
/// the live ranges of any instructions whose outputs are being used as /// the live ranges of any instructions whose outputs are being used as
/// operands to this instruction. /// operands to this instruction.
pub fn push_insn(&mut self, insn: Insn) { pub fn push_insn(&mut self, mut insn: Insn) {
// Index of this instruction // Index of this instruction
let insn_idx = self.insns.len(); let insn_idx = self.insns.len();
let mut insn = insn;
let mut opnd_iter = insn.opnd_iter_mut(); let mut opnd_iter = insn.opnd_iter_mut();
while let Some(opnd) = opnd_iter.next() { while let Some(opnd) = opnd_iter.next() {
match opnd { match opnd {
@ -1089,7 +1088,6 @@ impl Assembler
} }
// Set a side exit context to Target::SideExit // Set a side exit context to Target::SideExit
let mut insn = insn;
if let Some(Target::SideExit { context, .. }) = insn.target_mut() { if let Some(Target::SideExit { context, .. }) = insn.target_mut() {
// We should skip this when this instruction is being copied from another Assembler. // We should skip this when this instruction is being copied from another Assembler.
if context.is_none() { if context.is_none() {