FT->getParamType(2) != TD->getIntPtrType(*Context))
return 0;
- ConstantInt *SizeCI = dyn_cast<ConstantInt>(CI->getOperand(4));
- if (!SizeCI)
+ ConstantInt *ObjSizeCI = dyn_cast<ConstantInt>(CI->getOperand(4));
+ if (!ObjSizeCI)
return 0;
- if (SizeCI->isAllOnesValue()) {
+ ConstantInt *SizeCI = dyn_cast<ConstantInt>(CI->getOperand(3));
+ if (ObjSizeCI->isAllOnesValue() ||
+ (SizeCI && ObjSizeCI->getValue().uge(SizeCI->getValue()))) {
EmitMemCpy(CI->getOperand(1), CI->getOperand(2), CI->getOperand(3), 1, B);
return CI->getOperand(1);
}
FT->getParamType(2) != TD->getIntPtrType(*Context))
return 0;
- ConstantInt *SizeCI = dyn_cast<ConstantInt>(CI->getOperand(4));
- if (!SizeCI)
+ ConstantInt *ObjSizeCI = dyn_cast<ConstantInt>(CI->getOperand(4));
+ if (!ObjSizeCI)
return 0;
- if (SizeCI->isAllOnesValue()) {
+ ConstantInt *SizeCI = dyn_cast<ConstantInt>(CI->getOperand(3));
+ if (ObjSizeCI->isAllOnesValue() ||
+ (SizeCI && ObjSizeCI->getValue().uge(SizeCI->getValue()))) {
Value *Val = B.CreateIntCast(CI->getOperand(2), Type::getInt8Ty(*Context),
false);
EmitMemSet(CI->getOperand(1), Val, CI->getOperand(3), B);
FT->getParamType(2) != TD->getIntPtrType(*Context))
return 0;
- ConstantInt *SizeCI = dyn_cast<ConstantInt>(CI->getOperand(4));
- if (!SizeCI)
+ ConstantInt *ObjSizeCI = dyn_cast<ConstantInt>(CI->getOperand(4));
+ if (!ObjSizeCI)
return 0;
- if (SizeCI->isAllOnesValue()) {
+ ConstantInt *SizeCI = dyn_cast<ConstantInt>(CI->getOperand(3));
+ if (ObjSizeCI->isAllOnesValue() ||
+ (SizeCI && ObjSizeCI->getValue().uge(SizeCI->getValue()))) {
EmitMemMove(CI->getOperand(1), CI->getOperand(2), CI->getOperand(3),
1, B);
return CI->getOperand(1);
!FT->getParamType(1)->isPointerTy())
return 0;
- ConstantInt *SizeCI = dyn_cast<ConstantInt>(CI->getOperand(3));
- if (!SizeCI)
+ ConstantInt *ObjSizeCI = dyn_cast<ConstantInt>(CI->getOperand(3));
+ if (!ObjSizeCI)
return 0;
// If a) we don't have any length information, or b) we know this will
// strcpy_chk call which may fail at runtime if the size is too long.
// TODO: It might be nice to get a maximum length out of the possible
// string lengths for varying.
- if (SizeCI->isAllOnesValue() ||
- SizeCI->getZExtValue() >= GetStringLength(CI->getOperand(2)))
+ if (ObjSizeCI->isAllOnesValue() ||
+ ObjSizeCI->getZExtValue() >= GetStringLength(CI->getOperand(2)))
return EmitStrCpy(CI->getOperand(1), CI->getOperand(2), B);
return 0;
--- /dev/null
+; RUN: opt < %s -simplify-libcalls -S | FileCheck %s
+; rdar://7719085
+
+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-n8:16:32:64"
+
+%struct.data = type { [100 x i32], [100 x i32], [1024 x i8] }
+
+define i32 @t() nounwind ssp {
+; CHECK: @t
+; CHECK: @llvm.memset.i64
+entry:
+ %0 = alloca %struct.data, align 8 ; <%struct.data*> [#uses=1]
+ %1 = bitcast %struct.data* %0 to i8* ; <i8*> [#uses=1]
+ %2 = call i8* @__memset_chk(i8* %1, i32 0, i64 1824, i64 1824) nounwind ; <i8*> [#uses=0]
+ ret i32 0
+}
+
+declare i8* @__memset_chk(i8*, i32, i64, i64) nounwind