No need to verify that malloc's return type is i8*.
authorVictor Hernandez <vhernandez@apple.com>
Tue, 22 Sep 2009 18:50:03 +0000 (18:50 +0000)
committerVictor Hernandez <vhernandez@apple.com>
Tue, 22 Sep 2009 18:50:03 +0000 (18:50 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@82561 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Analysis/MallocHelper.cpp
lib/VMCore/Verifier.cpp

index 9c1671d306416b0f40571a0102394a959b18c6dd..ab6239e55ad433ec651e00f2ac90fd2bf23b476d 100644 (file)
@@ -155,7 +155,7 @@ const PointerType* llvm::getMallocType(const CallInst* CI) {
   if (BCI && CI->hasOneUse())
     return cast<PointerType>(BCI->getDestTy());
 
-  // Malloc call was not bitcast, so the type is the malloc's return type, i8*.
+  // Malloc call was not bitcast, so type is the malloc function's return type.
   if (!BCI)
     return cast<PointerType>(CI->getType());
 
index f1f6e2e199bf1ccbce2d3e75ad569741900d9ba1..140e6bd8b15a78c1ffeefa091b28afc4247bc77a 100644 (file)
@@ -1143,16 +1143,6 @@ void Verifier::visitCallInst(CallInst &CI) {
   if (Function *F = CI.getCalledFunction())
     if (Intrinsic::ID ID = (Intrinsic::ID)F->getIntrinsicID())
       visitIntrinsicFunctionCall(ID, CI);
-
-  // Code here matches isMalloc from MallocHelper, which is not in VMCore.
-  const Module* M = CI.getParent()->getParent()->getParent();
-  Constant *MallocFunc = M->getFunction("malloc");
-
-  if (CI.getOperand(0) == MallocFunc) {
-    const PointerType *PTy =
-        PointerType::getUnqual(Type::getInt8Ty(CI.getParent()->getContext()));
-    Assert1(CI.getType() == PTy, "Malloc call must return i8*", &CI);
-  }
 }
 
 void Verifier::visitInvokeInst(InvokeInst &II) {