commit d00bfe1f8dded563fa32a045d6d74ec1c25427dd
author Bruno Cardoso Lopes <bruno.cardoso@gmail.com> Thu Sep 02 02:35:51 2010 +0000
committer Bruno Cardoso Lopes <bruno.cardoso@gmail.com> Thu Sep 02 02:35:51 2010 +0000
tree a79b38ef72e82917c4a80cd48d9b74132291537c
parent 1f58741aabd00545468f4ecf29db3dba8e0bd6ac

become more strict about when it's safe to use X86ISD::MOVLPS

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@112799 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/X86/X86ISelLowering.cpp

diff --git a/lib/Target/X86/X86ISelLowering.cpp b/lib/Target/X86/X86ISelLowering.cpp
index 852d339..7acd893 100644
--- a/lib/Target/X86/X86ISelLowering.cpp
+++ b/lib/Target/X86/X86ISelLowering.cpp
@@ -5082,10 +5082,10 @@
   SDValue TmpV1 = V1;
   SDValue TmpV2 = V2;
 
-  // Trivial case, when V2 is a load.
-  if (TmpV2.getOpcode() == ISD::BIT_CONVERT)
+  // Trivial case, when V2 comes from a load.
+  if (TmpV2.hasOneUse() && TmpV2.getOpcode() == ISD::BIT_CONVERT)
     TmpV2 = TmpV2.getOperand(0);
-  if (TmpV2.getOpcode() == ISD::SCALAR_TO_VECTOR)
+  if (TmpV2.hasOneUse() && TmpV2.getOpcode() == ISD::SCALAR_TO_VECTOR)
     TmpV2 = TmpV2.getOperand(0);
   if (MayFoldLoad(TmpV2))
     CanFoldLoad = true;
@@ -5095,9 +5095,9 @@
   //    turns into:
   //  (MOVLPSmr addr:$src1, VR128:$src2)
   // So, recognize this potential and also use MOVLPS or MOVLPD
-  if (TmpV1.getOpcode() == ISD::BIT_CONVERT)
+  if (TmpV1.hasOneUse() && TmpV1.getOpcode() == ISD::BIT_CONVERT)
     TmpV1 = TmpV1.getOperand(0);
-  if (MayFoldLoad(TmpV1))
+  if (MayFoldLoad(TmpV1) && MayFoldIntoStore(Op))
     CanFoldLoad = true;
 
   if (CanFoldLoad) {