Hi,

could a gatekeeper review this patch?

Such a combination is abandoned for load-execution as the ld_op is a
load up higher part of a SSE register, while the alu_op is doing
conversion on the lower part (implicitly).

Best Regards,
Yiran

Index: osprey/be/cg/x8664/ebo_special.cxx
===================================================================
--- osprey/be/cg/x8664/ebo_special.cxx  (revision 1278)
+++ osprey/be/cg/x8664/ebo_special.cxx  (revision 1279)
@@ -9350,6 +9350,12 @@
     return Process_Side_Effects(opnd_tn, actual_tninfo, rval, opnds_swapped);
   }

+  if ((OP_code(ld_op) == TOP_ldhps ||
+       OP_code(ld_op) == TOP_ldhpsx ||
+       OP_code(ld_op) == TOP_ldhpsxx) &&
+      OP_code(alu_op) == TOP_cvtps2pd)
+    return FALSE;
+
   /* Check <index> and <base> will not be re-defined between
      <ld_op> and <alu_op>, inclusive.
   */

------------------------------------------------------------------------------
Forrester Wave Report - Recovery time is now measured in hours and minutes
not days. Key insights are discussed in the 2010 Forrester Wave Report as
part of an in-depth evaluation of disaster recovery service providers.
Forrester found the best-in-class provider in terms of services and vision.
Read this report now!  http://p.sf.net/sfu/ibm-webcastpromo
_______________________________________________
Open64-devel mailing list
Open64-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/open64-devel

Reply via email to