]> granicus.if.org Git - clang/commitdiff
fix a crash on code that uses the result value of __builtin___memcpy_chk.
authorChris Lattner <sabre@nondot.org>
Wed, 20 Apr 2011 23:14:50 +0000 (23:14 +0000)
committerChris Lattner <sabre@nondot.org>
Wed, 20 Apr 2011 23:14:50 +0000 (23:14 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@129892 91177308-0d34-0410-b5e6-96231b3b80d8

lib/CodeGen/CGBuiltin.cpp
test/CodeGen/builtin-memfns.c

index 51d9eebaba14be79461b609fab5f5134f96bba94..e553eb6c27880fb538da4d1386ef01d3e118ba6e 100644 (file)
@@ -556,7 +556,7 @@ RValue CodeGenFunction::EmitBuiltinExpr(const FunctionDecl *FD,
     Value *Src = EmitScalarExpr(E->getArg(1));
     Value *SizeVal = llvm::ConstantInt::get(Builder.getContext(), Size);
     Builder.CreateMemCpy(Dest, Src, SizeVal, 1, false);
-    return RValue::get(0);
+    return RValue::get(Dest);
   }
       
   case Builtin::BI__builtin_objc_memmove_collectable: {
@@ -581,7 +581,7 @@ RValue CodeGenFunction::EmitBuiltinExpr(const FunctionDecl *FD,
     Value *Src = EmitScalarExpr(E->getArg(1));
     Value *SizeVal = llvm::ConstantInt::get(Builder.getContext(), Size);
     Builder.CreateMemMove(Dest, Src, SizeVal, 1, false);
-    return RValue::get(0);
+    return RValue::get(Dest);
   }
 
   case Builtin::BImemmove:
@@ -616,7 +616,7 @@ RValue CodeGenFunction::EmitBuiltinExpr(const FunctionDecl *FD,
     Value *SizeVal = llvm::ConstantInt::get(Builder.getContext(), Size);
     Builder.CreateMemSet(Address, ByteVal, SizeVal, 1, false);
     
-    return RValue::get(0);
+    return RValue::get(Address);
   }
   case Builtin::BI__builtin_dwarf_cfa: {
     // The offset in bytes from the first argument to the CFA.
index 2ea6793654733103a8571e75f11cab7351f3c706..fb4d7200752de9a26052ed29f3c357bfcfc8fee3 100644 (file)
@@ -42,3 +42,9 @@ void test4(char *P, char *Q) {
 void test5(char *P, char *Q) {
   __builtin___memmove_chk(P, Q, 128, 128);
 }
+
+// CHECK: @test6
+// CHECK: call void @llvm.memcpy
+int test6(char *X) {
+  return __builtin___memcpy_chk(X, X, 42, 42) != 0;
+}