[PATCH] D18405: [PPC] Prefer floating point load to integer load plus direct move, when there is no other user for the integer load

Chad Rosier via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 5 12:38:46 PDT 2016


mcrosier added inline comments.

================
Comment at: lib/Target/PowerPC/PPCISelLowering.cpp:6389-6390
@@ -6364,3 +6388,4 @@
   // however, without FPCVT we can't do most conversions.
-  if (Subtarget.hasDirectMove() && Subtarget.isPPC64() && Subtarget.hasFPCVT())
+  if (Subtarget.hasDirectMove() && directMoveIsProfitable(Op) &&
+      Subtarget.isPPC64() && Subtarget.hasFPCVT())
     return LowerINT_TO_FPDirectMove(Op, DAG, dl);
----------------
Fair enough. :)


http://reviews.llvm.org/D18405





More information about the llvm-commits mailing list