From: Andrew Waterman Date: Thu, 27 Jan 2011 02:05:11 +0000 (-0800) Subject: [sim] changed divide-by-0 semantics X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=dc1aa62411649b73f3e3fb2ee5ec425a2bfeb645;p=riscv-isa-sim.git [sim] changed divide-by-0 semantics now it always gives -1, no matter the signedness. --- diff --git a/riscv/insns/div.h b/riscv/insns/div.h index aa67134..96d9e10 100644 --- a/riscv/insns/div.h +++ b/riscv/insns/div.h @@ -1,9 +1,6 @@ -if(RS2 == 0 || (sreg_t(RS1) == INT64_MIN && sreg_t(RS2) == -1)) -{ - if(xpr64) - RD = sreg_t(RS1) < 0 ? INT64_MIN : INT64_MAX; - else - RD = sreg_t(RS1) < 0 ? sext32(INT32_MIN) : INT32_MAX; -} +if(RS2 == 0) + RD = UINT64_MAX; +else if(sreg_t(RS1) == INT64_MIN && sreg_t(RS2) == -1) + RD = RS1; else RD = sext_xprlen(sreg_t(RS1) / sreg_t(RS2)); diff --git a/riscv/insns/divw.h b/riscv/insns/divw.h index 9c732b5..ee940da 100644 --- a/riscv/insns/divw.h +++ b/riscv/insns/divw.h @@ -1,7 +1,9 @@ require_xpr64; +if(RS2 == 0) + RD = UINT64_MAX; // INT64_MIN/-1 corner case shouldn't occur in correct code, since // INT64_MIN is not a proper 32-bit signed value -if(RS2 == 0 || (sreg_t(RS1) == INT64_MIN && sreg_t(RS2) == -1)) - RD = sext32(sreg_t(RS1) < 0 ? INT32_MIN : INT32_MAX); +else if(sreg_t(RS1) == INT64_MIN && sreg_t(RS2) == -1) + RD = RS1; else RD = sext32(sreg_t(RS1) / sreg_t(RS2));