From bbc8a40857d47348753c59b91a93dd312fa1b809 Mon Sep 17 00:00:00 2001 From: Gabe Black Date: Thu, 13 Sep 2007 16:35:41 -0700 Subject: [PATCH] X86: Fix how ECF is computed in genFlags, and get rid of some duplicate code. --HG-- extra : convert_revision : f86330a5a9fea782ee63aaa18ca964fb6f9cef0b --- src/arch/x86/insts/microregop.cc | 14 +++++--------- src/arch/x86/isa/insts/processor_information.py | 2 +- 2 files changed, 6 insertions(+), 10 deletions(-) diff --git a/src/arch/x86/insts/microregop.cc b/src/arch/x86/insts/microregop.cc index b6a30d6a7..ad48a4bc1 100644 --- a/src/arch/x86/insts/microregop.cc +++ b/src/arch/x86/insts/microregop.cc @@ -68,17 +68,15 @@ namespace X86ISA { DPRINTF(Sparc, "flagMask = %#x\n", flagMask); uint64_t flags = oldFlags & ~flagMask; - if(flagMask & CFBit) + if(flagMask & (ECFBit | CFBit)) { if(findCarry(dataSize*8, _dest, _src1, _src2)) - flags |= CFBit; + flags |= (flagMask & (ECFBit | CFBit)); if(subtract) - flags ^= CFBit; + flags ^= (flagMask & (ECFBit | CFBit)); } if(flagMask & PFBit && findParity(dataSize*8, _dest)) flags |= PFBit; - if(flagMask & ECFBit && findCarry(dataSize*8, _dest, _src1, _src2)) - flags |= ECFBit; if(flagMask & AFBit) { if(findCarry(4, _dest, _src1, _src2)) @@ -86,10 +84,8 @@ namespace X86ISA if(subtract) flags ^= AFBit; } - if(flagMask & EZFBit && findZero(dataSize*8, _dest)) - flags |= EZFBit; - if(flagMask & ZFBit && findZero(dataSize*8, _dest)) - flags |= ZFBit; + if(flagMask & (EZFBit | ZFBit) && findZero(dataSize*8, _dest)) + flags |= (flagMask & (EZFBit | ZFBit)); if(flagMask & SFBit && findNegative(dataSize*8, _dest)) flags |= SFBit; if(flagMask & OFBit && findOverflow(dataSize*8, _dest, _src1, _src2)) diff --git a/src/arch/x86/isa/insts/processor_information.py b/src/arch/x86/isa/insts/processor_information.py index 9cad8181c..6070169ac 100644 --- a/src/arch/x86/isa/insts/processor_information.py +++ b/src/arch/x86/isa/insts/processor_information.py @@ -74,7 +74,7 @@ def macroop CPUID_R { movi t2, t2, 0x18, flags=(nCEZF,) subi t0, t1, t2, flags=(ECF,) # ECF will be set if the offset is too large. - bri t0, label("end"), flags=(CECF,) + bri t0, label("end"), flags=(nCECF,) # -- 2.30.2