From 15c821a2d3fbf2fc0458090b6cc12f2ac093f075 Mon Sep 17 00:00:00 2001 From: iximeow Date: Sun, 9 Jul 2023 08:41:46 -0700 Subject: restructuring of hotpath code, not worse but not better --- src/long_mode/mod.rs | 32 ++++++++++++++++++-------------- 1 file changed, 18 insertions(+), 14 deletions(-) (limited to 'src/long_mode') diff --git a/src/long_mode/mod.rs b/src/long_mode/mod.rs index 533efa4..4d5cf59 100644 --- a/src/long_mode/mod.rs +++ b/src/long_mode/mod.rs @@ -6466,7 +6466,9 @@ impl DecodeCtx { fn read_opc_hotpath< T: Reader<::Address, ::Word>, S: DescriptionSink, ->(&mut self, mut b: u8, nextb: &mut u8, record: &mut OpcodeRecord, words: &mut T, instruction: &mut Instruction, sink: &mut S) -> Result { +>(&mut self, words: &mut T, instruction: &mut Instruction, sink: &mut S) -> Result, DecodeError> { + let mut b = words.next().ok().ok_or(DecodeError::ExhaustedInput)?; + if b >= 0x40 && b < 0x50 { sink.record((words.offset() - 1) as u32 * 8, (words.offset() - 1) as u32 * 8 + 7, FieldDescription { desc: InnerDescription::RexPrefix(b), @@ -6478,7 +6480,7 @@ fn read_opc_hotpath< } self.rb_size = RegisterBank::rB; b = words.next().ok().ok_or(DecodeError::ExhaustedInput)?; - *record = unsafe { + record = unsafe { core::ptr::read_volatile(&OPCODES[b as usize]) }; } else if b == 0x66 { @@ -6487,20 +6489,24 @@ fn read_opc_hotpath< id: words.offset() as u32 * 8 - 8, }); b = words.next().ok().ok_or(DecodeError::ExhaustedInput)?; - *record = unsafe { + record = unsafe { core::ptr::read_volatile(&OPCODES[b as usize]) }; instruction.prefixes.set_operand_size(); self.vqp_size = RegisterBank::W; + } else { + record = unsafe { + core::ptr::read_volatile(&OPCODES[b as usize]) + }; } if let Interpretation::Instruction(opc) = record.0 { record_opcode_record_found(words, sink, opc, record.1, 1); instruction.opcode = opc; - return Ok(true); + return Ok(Ok(record.1)); } else if b == 0x0f { let b = words.next().ok().ok_or(DecodeError::ExhaustedInput)?; - let (r, len) = if b == 0x38 { + let (record, len) = if b == 0x38 { let b = words.next().ok().ok_or(DecodeError::ExhaustedInput)?; (self.read_0f38_opcode(b, &mut instruction.prefixes), 3) } else if b == 0x3a { @@ -6509,17 +6515,15 @@ fn read_opc_hotpath< } else { (self.read_0f_opcode(b, &mut instruction.prefixes), 2) }; - *record = r; if let Interpretation::Instruction(opc) = record.0 { record_opcode_record_found(words, sink, opc, record.1, len); instruction.opcode = opc; } else { unsafe { unreachable_unchecked(); } } - return Ok(true); + return Ok(Ok(record.1)); } else { - *nextb = b; - return Ok(false); + return Ok(Err((b, record))); } } @@ -6529,8 +6533,6 @@ fn read_with_annotations< S: DescriptionSink, >(mut self, decoder: &InstDecoder, words: &mut T, instruction: &mut Instruction, sink: &mut S) -> Result<(), DecodeError> { words.mark(); - let mut nextb = words.next().ok().ok_or(DecodeError::ExhaustedInput)?; - let mut next_rec = OPCODES[nextb as usize]; instruction.prefixes = Prefixes::new(0); // const RAXRAXRAXRAX: [RegSpec; 4] = [RegSpec::rax(); 4]; @@ -6539,9 +6541,10 @@ fn read_with_annotations< instruction.regs[1] = RegSpec::rax(); instruction.regs[2] = RegSpec::rax(); - let record: OperandCode = if self.read_opc_hotpath(nextb, &mut nextb, &mut next_rec, words, instruction, sink)? { - next_rec.1 - } else { + let hotpath_res = self.read_opc_hotpath(words, instruction, sink)?; + let record: OperandCode = match hotpath_res { + Ok(code) => code, + Err((mut nextb, mut next_rec)) => { let prefixes = &mut instruction.prefixes; let record = loop { let mut record = next_rec; @@ -6684,6 +6687,7 @@ fn read_with_annotations< } record.1 + } }; self.read_operands(decoder, words, instruction, record, sink)?; -- cgit v1.1