|
From: <sv...@va...> - 2006-01-01 13:17:48
|
Author: sewardj
Date: 2006-01-01 13:17:38 +0000 (Sun, 01 Jan 2006)
New Revision: 1521
Log:
For SSE scalar comparison operations where one operand is in memory,
do not read more memory than needed.
Modified:
trunk/priv/guest-amd64/toIR.c
Modified: trunk/priv/guest-amd64/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-amd64/toIR.c 2005-12-30 15:04:29 UTC (rev 1520)
+++ trunk/priv/guest-amd64/toIR.c 2006-01-01 13:17:38 UTC (rev 1521)
@@ -7601,7 +7601,7 @@
vpanic("findSSECmpOp(amd64,guest)");
}
=20
-/* Handles SSE 32F comparisons. */
+/* Handles SSE 32F/64F comparisons. */
=20
static ULong dis_SSEcmp_E_to_G ( Prefix pfx, Long delta,=20
HChar* opname, Bool all_lanes, Int sz )
@@ -7629,8 +7629,15 @@
addr =3D disAMode ( &alen, pfx, delta, dis_buf, 1 );
imm8 =3D getUChar(delta+alen);
findSSECmpOp(&needNot, &op, imm8, all_lanes, sz);
- assign( plain, binop(op, getXMMReg(gregOfRexRM(pfx,rm)),=20
- loadLE(Ity_V128, mkexpr(addr))) );
+ assign( plain,=20
+ binop(
+ op,
+ getXMMReg(gregOfRexRM(pfx,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,
|