|
From: <sv...@va...> - 2006-01-01 17:15:24
|
Author: sewardj
Date: 2006-01-01 17:15:19 +0000 (Sun, 01 Jan 2006)
New Revision: 1522
Log:
x86 counterpart to r1521: For SSE scalar comparison operations where
one operand is in memory, do not read more memory than needed.
Modified:
trunk/priv/guest-x86/toIR.c
Modified: trunk/priv/guest-x86/toIR.c
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
--- trunk/priv/guest-x86/toIR.c 2006-01-01 13:17:38 UTC (rev 1521)
+++ trunk/priv/guest-x86/toIR.c 2006-01-01 17:15:19 UTC (rev 1522)
@@ -6672,7 +6672,7 @@
vpanic("findSSECmpOp(x86,guest)");
}
=20
-/* Handles SSE 32F comparisons. */
+/* Handles SSE 32F/64F comparisons. */
=20
static UInt dis_SSEcmp_E_to_G ( UChar sorb, Int delta,=20
HChar* opname, Bool all_lanes, Int sz )
@@ -6700,8 +6700,15 @@
addr =3D disAMode ( &alen, sorb, delta, dis_buf );
imm8 =3D getIByte(delta+alen);
findSSECmpOp(&needNot, &op, imm8, all_lanes, sz);
- assign( plain, binop(op, getXMMReg(gregOfRM(rm)),=20
- loadLE(Ity_V128, mkexpr(addr))) );
+ assign( plain,=20
+ binop(
+ op,
+ getXMMReg(gregOfRM(rm)),=20
+ all_lanes ? loadLE(Ity_V128, mkexpr(addr))
+ : sz =3D=3D 8 ? unop( Iop_64UtoV128, loadLE(Ity_I64,=
mkexpr(addr)))
+ : /*sz=3D=3D4*/ unop( Iop_32UtoV128, loadLE(Ity_I32,=
mkexpr(addr)))
+ )=20
+ );
delta +=3D alen+1;
DIP("%s $%d,%s,%s\n", opname,
(Int)imm8,
|