From 98e460c30d2f5e3848d7ae429bebcce3b8c90b81 Mon Sep 17 00:00:00 2001 From: DJ Delorie Date: Wed, 6 Feb 2008 04:41:26 +0000 Subject: [PATCH] * simops.c (OP_1C007E0): Compensate for 64 bit hosts. (OP_18007E0): Likewise. (OP_2C007E0): Likewise. (OP_28007E0): Likewise. * v850.igen (divh): Likewise. --- sim/v850/ChangeLog | 6 ++++++ sim/v850/simops.c | 16 +++++++++------- sim/v850/v850.igen | 4 ++-- 3 files changed, 17 insertions(+), 9 deletions(-) diff --git a/sim/v850/ChangeLog b/sim/v850/ChangeLog index 536d97929cb..c3b3874c3c3 100644 --- a/sim/v850/ChangeLog +++ b/sim/v850/ChangeLog @@ -1,5 +1,11 @@ 2008-02-05 DJ Delorie + * simops.c (OP_1C007E0): Compensate for 64 bit hosts. + (OP_18007E0): Likewise. + (OP_2C007E0): Likewise. + (OP_28007E0): Likewise. + * v850.igen (divh): Likewise. + * simops.c (OP_C0): Correct saturation logic. (OP_220): Likewise. (OP_A0): Likewise. diff --git a/sim/v850/simops.c b/sim/v850/simops.c index 551334175c7..49f12004cfe 100644 --- a/sim/v850/simops.c +++ b/sim/v850/simops.c @@ -2209,8 +2209,8 @@ OP_1C007E0 (void) imm5 = 32 - ((OP[3] & 0x3c0000) >> 17); - divide_by = State.regs[ OP[0] ]; - divide_this = State.regs[ OP[1] ] << imm5; + divide_by = (signed32) State.regs[ OP[0] ]; + divide_this = (signed32) (State.regs[ OP[1] ] << imm5); divn (imm5, divide_by, divide_this, & quotient, & remainder, & overflow); @@ -2280,7 +2280,7 @@ OP_18007E0 (void) imm5 = 32 - ((OP[3] & 0x3c0000) >> 17); divide_by = EXTEND16 (State.regs[ OP[0] ]); - divide_this = State.regs[ OP[1] ] << imm5; + divide_this = (signed32) (State.regs[ OP[1] ] << imm5); divn (imm5, divide_by, divide_this, & quotient, & remainder, & overflow); @@ -2351,14 +2351,14 @@ OP_2C007E0 (void) /* Compute the result. */ - divide_by = State.regs[ OP[0] ]; + divide_by = (signed32) State.regs[ OP[0] ]; divide_this = State.regs[ OP[1] ]; if (divide_by == 0) { PSW |= PSW_OV; } - else if (divide_by == -1 && divide_this == (1 << 31)) + else if (divide_by == -1 && divide_this == (1L << 31)) { PSW &= ~PSW_Z; PSW |= PSW_OV | PSW_S; @@ -2367,9 +2367,10 @@ OP_2C007E0 (void) } else { + divide_this = (signed32) divide_this; State.regs[ OP[1] ] = quotient = divide_this / divide_by; State.regs[ OP[2] >> 11 ] = remainder = divide_this % divide_by; - + /* Set condition codes. */ PSW &= ~(PSW_Z | PSW_S | PSW_OV); @@ -2442,7 +2443,7 @@ OP_28007E0 (void) { PSW |= PSW_OV; } - else if (divide_by == -1 && divide_this == (1 << 31)) + else if (divide_by == -1 && divide_this == (1L << 31)) { PSW &= ~PSW_Z; PSW |= PSW_OV | PSW_S; @@ -2451,6 +2452,7 @@ OP_28007E0 (void) } else { + divide_this = (signed32) divide_this; State.regs[ OP[1] ] = quotient = divide_this / divide_by; State.regs[ OP[2] >> 11 ] = remainder = divide_this % divide_by; diff --git a/sim/v850/v850.igen b/sim/v850/v850.igen index 6617bd85659..c0382bce913 100644 --- a/sim/v850/v850.igen +++ b/sim/v850/v850.igen @@ -356,7 +356,7 @@ rrrrr!0,000010,RRRRR!0:I:::divh op0 = EXTEND16 (State.regs[OP[0]]); op1 = State.regs[OP[1]]; - if (op0 == 0xffffffff && op1 == 0x80000000) + if (op0 == -1 && op1 == 0x80000000) { PSW &= ~PSW_Z; PSW |= PSW_OV | PSW_S; @@ -368,7 +368,7 @@ rrrrr!0,000010,RRRRR!0:I:::divh } else { - result = op1 / op0; + result = (signed32) op1 / op0; ov = 0; /* Compute the condition codes. */ -- 2.30.2