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