|
 |
560544 |
commit 9545e9f96beda6e9f2205bdb3c3e96edaf8d9e2b
|
|
|
560544 |
Author: Andreas Arnez <arnez@linux.ibm.com>
|
|
|
560544 |
Date: Tue Oct 30 17:06:38 2018 +0100
|
|
|
560544 |
|
|
|
560544 |
Bug 400491 s390x: Sign-extend immediate operand of LOCHI and friends
|
|
|
560544 |
|
|
|
560544 |
The VEX implementation of each of the z/Architecture instructions LOCHI,
|
|
|
560544 |
LOCHHI, and LOCGHI treats the immediate 16-bit operand as an unsigned
|
|
|
560544 |
integer instead of a signed integer. This is fixed.
|
|
|
560544 |
|
|
|
560544 |
diff --git a/VEX/priv/guest_s390_toIR.c b/VEX/priv/guest_s390_toIR.c
|
|
|
560544 |
index 60b6081..9c4d79b 100644
|
|
|
560544 |
--- a/VEX/priv/guest_s390_toIR.c
|
|
|
560544 |
+++ b/VEX/priv/guest_s390_toIR.c
|
|
|
560544 |
@@ -16307,7 +16307,7 @@ static const HChar *
|
|
|
560544 |
s390_irgen_LOCHHI(UChar r1, UChar m3, UShort i2, UChar unused)
|
|
|
560544 |
{
|
|
|
560544 |
next_insn_if(binop(Iop_CmpEQ32, s390_call_calculate_cond(m3), mkU32(0)));
|
|
|
560544 |
- put_gpr_w0(r1, mkU32(i2));
|
|
|
560544 |
+ put_gpr_w0(r1, mkU32((UInt)(Int)(Short)i2));
|
|
|
560544 |
|
|
|
560544 |
return "lochhi";
|
|
|
560544 |
}
|
|
|
560544 |
@@ -16316,7 +16316,7 @@ static const HChar *
|
|
|
560544 |
s390_irgen_LOCHI(UChar r1, UChar m3, UShort i2, UChar unused)
|
|
|
560544 |
{
|
|
|
560544 |
next_insn_if(binop(Iop_CmpEQ32, s390_call_calculate_cond(m3), mkU32(0)));
|
|
|
560544 |
- put_gpr_w1(r1, mkU32(i2));
|
|
|
560544 |
+ put_gpr_w1(r1, mkU32((UInt)(Int)(Short)i2));
|
|
|
560544 |
|
|
|
560544 |
return "lochi";
|
|
|
560544 |
}
|
|
|
560544 |
@@ -16325,7 +16325,7 @@ static const HChar *
|
|
|
560544 |
s390_irgen_LOCGHI(UChar r1, UChar m3, UShort i2, UChar unused)
|
|
|
560544 |
{
|
|
|
560544 |
next_insn_if(binop(Iop_CmpEQ32, s390_call_calculate_cond(m3), mkU32(0)));
|
|
|
560544 |
- put_gpr_dw0(r1, mkU64(i2));
|
|
|
560544 |
+ put_gpr_dw0(r1, mkU64((UInt)(Int)(Short)i2));
|
|
|
560544 |
|
|
|
560544 |
return "locghi";
|
|
|
560544 |
}
|