From dad30582248b049a72dd3aa0350f19b1b8c456ee Mon Sep 17 00:00:00 2001 From: Gabe Black Date: Thu, 14 Jun 2007 20:52:24 +0000 Subject: [PATCH] Sign extend byte immediates as well. There might need to be a fancier system in place to handle this in the future. --HG-- extra : convert_revision : 2c5bd719c770d9a93a57bd29782842f82384863d --- src/arch/x86/predecoder.cc | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/src/arch/x86/predecoder.cc b/src/arch/x86/predecoder.cc index 72b60385b..3ed18aeb2 100644 --- a/src/arch/x86/predecoder.cc +++ b/src/arch/x86/predecoder.cc @@ -390,8 +390,15 @@ namespace X86ISA //Instructions which use true 64 bit immediates won't be //affected, and instructions that use true 32 bit immediates //won't notice. - if(immediateSize == 4) + switch(immediateSize) + { + case 4: emi.immediate = sext<32>(emi.immediate); + break; + case 1: + emi.immediate = sext<8>(emi.immediate); + } + DPRINTF(Predecoder, "Collected immediate %#x.\n", emi.immediate); emiIsReady = true; -- 2.30.2