Fix simplifylibcalls memset recognition to work on 64-bit platforms
authorEli Friedman <eli.friedman@gmail.com>
Sat, 18 Jul 2009 08:34:51 +0000 (08:34 +0000)
committerEli Friedman <eli.friedman@gmail.com>
Sat, 18 Jul 2009 08:34:51 +0000 (08:34 +0000)
where int is 32 bits.

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

lib/Transforms/Scalar/SimplifyLibCalls.cpp
test/Transforms/SimplifyLibCalls/memset-64.ll [new file with mode: 0644]

index f285406b19022c733b9bb9e55b05692dea8a1782..7e6b35346259e03e0d3165037a12564dde81ae5b 100644 (file)
@@ -994,12 +994,12 @@ struct VISIBILITY_HIDDEN MemSetOpt : public LibCallOptimization {
     const FunctionType *FT = Callee->getFunctionType();
     if (FT->getNumParams() != 3 || FT->getReturnType() != FT->getParamType(0) ||
         !isa<PointerType>(FT->getParamType(0)) ||
-        FT->getParamType(1) != TD->getIntPtrType() ||
+        !isa<IntegerType>(FT->getParamType(1)) ||
         FT->getParamType(2) != TD->getIntPtrType())
       return 0;
 
     // memset(p, v, n) -> llvm.memset(p, v, n, 1)
-    Value *Val = B.CreateTrunc(CI->getOperand(2), Type::Int8Ty);
+    Value *Val = B.CreateIntCast(CI->getOperand(2), Type::Int8Ty, false);
     EmitMemSet(CI->getOperand(1), Val,  CI->getOperand(3), B);
     return CI->getOperand(1);
   }
diff --git a/test/Transforms/SimplifyLibCalls/memset-64.ll b/test/Transforms/SimplifyLibCalls/memset-64.ll
new file mode 100644 (file)
index 0000000..a6a277e
--- /dev/null
@@ -0,0 +1,12 @@
+; RUN: llvm-as < %s | opt -simplify-libcalls | llvm-dis | grep {llvm.memset}
+target datalayout = "e-p:64:64:64-i1:8:8-i8:8:8-i16:16:16-i32:32:32-i64:64:64-f32:32:32-f64:64:64-v64:64:64-v128:128:128-a0:0:64-s0:64:64-f80:128:128"
+target triple = "x86_64-pc-linux-gnu"
+
+define void @a(i8* %x) nounwind {
+entry:
+       %call = call i8* @memset(i8* %x, i32 1, i64 100)                ; <i8*> [#uses=0]
+       ret void
+}
+
+declare i8* @memset(i8*, i32, i64)
+