[llvm-commits] [dragonegg] r103622 - /dragonegg/trunk/x86/llvm-target.cpp

[Duncan Sands]

Author: baldrick
Date: Wed May 12 12:10:37 2010
New Revision: 103622

URL: http://llvm.org/viewvc/llvm-project?rev=103622&view=rev
Log:
TargetIntrinsicLower is only supposed to set Result if there is a
value to be returned, so it is wrong to set it to a StoreInst for
example.  Fix a bunch of mistakes of this kind.

Modified:
    dragonegg/trunk/x86/llvm-target.cpp

Modified: dragonegg/trunk/x86/llvm-target.cpp
URL: http://llvm.org/viewvc/llvm-project/dragonegg/trunk/x86/llvm-target.cpp?rev=103622&r1=103621&r2=103622&view=diff
==============================================================================
--- dragonegg/trunk/x86/llvm-target.cpp (original)
+++ dragonegg/trunk/x86/llvm-target.cpp Wed May 12 12:10:37 2010
@@ -1036,7 +1036,6 @@
     Value *BC = Builder.CreateBitCast(Ops[0], v4f32Ptr);
     StoreInst *SI = Builder.CreateStore(Ops[1], BC);
     SI->setAlignment(1);
-    Result = SI;
     return true;
   }
   IX86_BUILTIN_STOREUPD: {
@@ -1045,7 +1044,6 @@
     Value *BC = Builder.CreateBitCast(Ops[0], v2f64Ptr);
     StoreInst *SI = Builder.CreateStore(Ops[1], BC);
     SI->setAlignment(1);
-    Result = SI;
     return true;
   }
   IX86_BUILTIN_STOREDQU: {
@@ -1054,7 +1052,6 @@
     Value *BC = Builder.CreateBitCast(Ops[0], v16i8Ptr);
     StoreInst *SI = Builder.CreateStore(Ops[1], BC);
     SI->setAlignment(1);
-    Result = SI;
     return true;
   }
   IX86_BUILTIN_LOADHPS: {
@@ -1100,7 +1097,7 @@
     Value *Idx = ConstantInt::get(Type::getInt32Ty(Context), 1);
     Ops[1] = Builder.CreateBitCast(Ops[1], v2f64);
     Ops[1] = Builder.CreateExtractElement(Ops[1], Idx);
-    Result = Builder.CreateStore(Ops[1], Ops[0]);
+    Builder.CreateStore(Ops[1], Ops[0]);
     return true;
   }
   IX86_BUILTIN_STORELPS: {
@@ -1110,7 +1107,7 @@
     Value *Idx = ConstantInt::get(Type::getInt32Ty(Context), 0);
     Ops[1] = Builder.CreateBitCast(Ops[1], v2f64);
     Ops[1] = Builder.CreateExtractElement(Ops[1], Idx);
-    Result = Builder.CreateStore(Ops[1], Ops[0]);
+    Builder.CreateStore(Ops[1], Ops[0]);
     return true;
   }
   IX86_BUILTIN_MOVSHDUP:
@@ -1256,7 +1253,7 @@
     Value *Ptr = CreateTemporary(Type::getInt32Ty(Context));
     Builder.CreateStore(Ops[0], Ptr);
     Ptr = Builder.CreateBitCast(Ptr, Type::getInt8PtrTy(Context));
-    Result = Builder.CreateCall(ldmxcsr, Ptr);
+    Builder.CreateCall(ldmxcsr, Ptr);
     return true;
   }
   IX86_BUILTIN_STMXCSR: {