ruby-changes:73324
From: Alan <ko1@a...>
Date: Tue, 30 Aug 2022 01:10:23 +0900 (JST)
Subject: [ruby-changes:73324] 29bda0ff81 (master): Use shorter syntax for the same pattern (https://github.com/Shopify/ruby/pull/425)
https://git.ruby-lang.org/ruby.git/commit/?id=29bda0ff81 From 29bda0ff8124ccfe660bfde43b59acd825f2d014 Mon Sep 17 00:00:00 2001 From: Alan Wu <XrXr@u...> Date: Thu, 18 Aug 2022 15:41:41 -0400 Subject: Use shorter syntax for the same pattern (https://github.com/Shopify/ruby/pull/425) --- yjit/src/backend/ir.rs | 84 +++++++++++++++++++++++++------------------------- 1 file changed, 42 insertions(+), 42 deletions(-) diff --git a/yjit/src/backend/ir.rs b/yjit/src/backend/ir.rs index 4a7bea8dac..f01ab398da 100644 --- a/yjit/src/backend/ir.rs +++ b/yjit/src/backend/ir.rs @@ -662,27 +662,27 @@ impl<'a> Iterator for InsnOpndIterator<'a> { https://github.com/ruby/ruby/blob/trunk/yjit/src/backend/ir.rs#L662 _ => None } }, - Insn::Add { left: opnd0 @ _, right: opnd1 @ _, .. } | - Insn::And { left: opnd0 @ _, right: opnd1 @ _, .. } | - Insn::Cmp { left: opnd0 @ _, right: opnd1 @ _ } | - Insn::CSelE { truthy: opnd0 @ _, falsy: opnd1 @ _, .. } | - Insn::CSelG { truthy: opnd0 @ _, falsy: opnd1 @ _, .. } | - Insn::CSelGE { truthy: opnd0 @ _, falsy: opnd1 @ _, .. } | - Insn::CSelL { truthy: opnd0 @ _, falsy: opnd1 @ _, .. } | - Insn::CSelLE { truthy: opnd0 @ _, falsy: opnd1 @ _, .. } | - Insn::CSelNE { truthy: opnd0 @ _, falsy: opnd1 @ _, .. } | - Insn::CSelNZ { truthy: opnd0 @ _, falsy: opnd1 @ _, .. } | - Insn::CSelZ { truthy: opnd0 @ _, falsy: opnd1 @ _, .. } | - Insn::IncrCounter { mem: opnd0 @ _, value: opnd1 @ _, .. } | - Insn::LShift { opnd: opnd0 @ _, shift: opnd1 @ _, .. } | - Insn::Mov { dest: opnd0 @ _, src: opnd1 @ _ } | - Insn::Or { left: opnd0 @ _, right: opnd1 @ _, .. } | - Insn::RShift { opnd: opnd0 @ _, shift: opnd1 @ _, .. } | - Insn::Store { dest: opnd0 @ _, src: opnd1 @ _ } | - Insn::Sub { left: opnd0 @ _, right: opnd1 @ _, .. } | - Insn::Test { left: opnd0 @ _, right: opnd1 @ _ } | - Insn::URShift { opnd: opnd0 @ _, shift: opnd1 @ _, .. } | - Insn::Xor { left: opnd0 @ _, right: opnd1 @ _, .. } => { + Insn::Add { left: opnd0, right: opnd1, .. } | + Insn::And { left: opnd0, right: opnd1, .. } | + Insn::Cmp { left: opnd0, right: opnd1 } | + Insn::CSelE { truthy: opnd0, falsy: opnd1, .. } | + Insn::CSelG { truthy: opnd0, falsy: opnd1, .. } | + Insn::CSelGE { truthy: opnd0, falsy: opnd1, .. } | + Insn::CSelL { truthy: opnd0, falsy: opnd1, .. } | + Insn::CSelLE { truthy: opnd0, falsy: opnd1, .. } | + Insn::CSelNE { truthy: opnd0, falsy: opnd1, .. } | + Insn::CSelNZ { truthy: opnd0, falsy: opnd1, .. } | + Insn::CSelZ { truthy: opnd0, falsy: opnd1, .. } | + Insn::IncrCounter { mem: opnd0, value: opnd1, .. } | + Insn::LShift { opnd: opnd0, shift: opnd1, .. } | + Insn::Mov { dest: opnd0, src: opnd1 } | + Insn::Or { left: opnd0, right: opnd1, .. } | + Insn::RShift { opnd: opnd0, shift: opnd1, .. } | + Insn::Store { dest: opnd0, src: opnd1 } | + Insn::Sub { left: opnd0, right: opnd1, .. } | + Insn::Test { left: opnd0, right: opnd1 } | + Insn::URShift { opnd: opnd0, shift: opnd1, .. } | + Insn::Xor { left: opnd0, right: opnd1, .. } => { match self.idx { 0 => { self.idx += 1; @@ -758,27 +758,27 @@ impl<'a> InsnOpndMutIterator<'a> { https://github.com/ruby/ruby/blob/trunk/yjit/src/backend/ir.rs#L758 _ => None } }, - Insn::Add { left: opnd0 @ _, right: opnd1 @ _, .. } | - Insn::And { left: opnd0 @ _, right: opnd1 @ _, .. } | - Insn::Cmp { left: opnd0 @ _, right: opnd1 @ _ } | - Insn::CSelE { truthy: opnd0 @ _, falsy: opnd1 @ _, .. } | - Insn::CSelG { truthy: opnd0 @ _, falsy: opnd1 @ _, .. } | - Insn::CSelGE { truthy: opnd0 @ _, falsy: opnd1 @ _, .. } | - Insn::CSelL { truthy: opnd0 @ _, falsy: opnd1 @ _, .. } | - Insn::CSelLE { truthy: opnd0 @ _, falsy: opnd1 @ _, .. } | - Insn::CSelNE { truthy: opnd0 @ _, falsy: opnd1 @ _, .. } | - Insn::CSelNZ { truthy: opnd0 @ _, falsy: opnd1 @ _, .. } | - Insn::CSelZ { truthy: opnd0 @ _, falsy: opnd1 @ _, .. } | - Insn::IncrCounter { mem: opnd0 @ _, value: opnd1 @ _, .. } | - Insn::LShift { opnd: opnd0 @ _, shift: opnd1 @ _, .. } | - Insn::Mov { dest: opnd0 @ _, src: opnd1 @ _ } | - Insn::Or { left: opnd0 @ _, right: opnd1 @ _, .. } | - Insn::RShift { opnd: opnd0 @ _, shift: opnd1 @ _, .. } | - Insn::Store { dest: opnd0 @ _, src: opnd1 @ _ } | - Insn::Sub { left: opnd0 @ _, right: opnd1 @ _, .. } | - Insn::Test { left: opnd0 @ _, right: opnd1 @ _ } | - Insn::URShift { opnd: opnd0 @ _, shift: opnd1 @ _, .. } | - Insn::Xor { left: opnd0 @ _, right: opnd1 @ _, .. } => { + Insn::Add { left: opnd0, right: opnd1, .. } | + Insn::And { left: opnd0, right: opnd1, .. } | + Insn::Cmp { left: opnd0, right: opnd1 } | + Insn::CSelE { truthy: opnd0, falsy: opnd1, .. } | + Insn::CSelG { truthy: opnd0, falsy: opnd1, .. } | + Insn::CSelGE { truthy: opnd0, falsy: opnd1, .. } | + Insn::CSelL { truthy: opnd0, falsy: opnd1, .. } | + Insn::CSelLE { truthy: opnd0, falsy: opnd1, .. } | + Insn::CSelNE { truthy: opnd0, falsy: opnd1, .. } | + Insn::CSelNZ { truthy: opnd0, falsy: opnd1, .. } | + Insn::CSelZ { truthy: opnd0, falsy: opnd1, .. } | + Insn::IncrCounter { mem: opnd0, value: opnd1, .. } | + Insn::LShift { opnd: opnd0, shift: opnd1, .. } | + Insn::Mov { dest: opnd0, src: opnd1 } | + Insn::Or { left: opnd0, right: opnd1, .. } | + Insn::RShift { opnd: opnd0, shift: opnd1, .. } | + Insn::Store { dest: opnd0, src: opnd1 } | + Insn::Sub { left: opnd0, right: opnd1, .. } | + Insn::Test { left: opnd0, right: opnd1 } | + Insn::URShift { opnd: opnd0, shift: opnd1, .. } | + Insn::Xor { left: opnd0, right: opnd1, .. } => { match self.idx { 0 => { self.idx += 1; -- cgit v1.2.1 -- ML: ruby-changes@q... Info: http://www.atdot.net/~ko1/quickml/