From 421f488cb7ba0798bb696ae363c79b526df8d3f7 Mon Sep 17 00:00:00 2001 From: Nuno Lopes Date: Sat, 6 Sep 2008 17:44:06 +0000 Subject: [PATCH] fix crash when the malloc/free function is defined or is a declaration with 0 parameters. this pass doesnt seem to be used, but still it's now a little more correct llvm-svn: 55873 --- llvm/lib/Transforms/IPO/IndMemRemoval.cpp | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/llvm/lib/Transforms/IPO/IndMemRemoval.cpp b/llvm/lib/Transforms/IPO/IndMemRemoval.cpp index 2e09fe967f4f..b251ab4b09c1 100644 --- a/llvm/lib/Transforms/IPO/IndMemRemoval.cpp +++ b/llvm/lib/Transforms/IPO/IndMemRemoval.cpp @@ -51,8 +51,7 @@ bool IndMemRemPass::runOnModule(Module &M) { //happen through intrinsics. bool changed = false; if (Function* F = M.getFunction("free")) { - assert(F->isDeclaration() && "free not external?"); - if (!F->use_empty()) { + if (F->isDeclaration() && F->arg_size() == 1 && !F->use_empty()) { Function* FN = Function::Create(F->getFunctionType(), GlobalValue::LinkOnceLinkage, "free_llvm_bounce", &M); @@ -66,8 +65,7 @@ bool IndMemRemPass::runOnModule(Module &M) { } } if (Function* F = M.getFunction("malloc")) { - assert(F->isDeclaration() && "malloc not external?"); - if (!F->use_empty()) { + if (F->isDeclaration() && F->arg_size() == 1 && !F->use_empty()) { Function* FN = Function::Create(F->getFunctionType(), GlobalValue::LinkOnceLinkage, "malloc_llvm_bounce", &M); -- GitLab