From fd595203bd0d501fae9c1341e8424b9f86af2bb8 Mon Sep 17 00:00:00 2001 From: Chris Lattner Date: Fri, 14 Feb 2003 19:12:29 +0000 Subject: [PATCH] Fix a bug that cause a crash resolving questionable function calls git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@5567 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Transforms/IPO/FunctionResolution.cpp | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/lib/Transforms/IPO/FunctionResolution.cpp b/lib/Transforms/IPO/FunctionResolution.cpp index 9c5580b3b26..3d5991aba98 100644 --- a/lib/Transforms/IPO/FunctionResolution.cpp +++ b/lib/Transforms/IPO/FunctionResolution.cpp @@ -49,8 +49,8 @@ static void ConvertCallTo(CallInst *CI, Function *Dest) { // BasicBlock::iterator BBI = CI; unsigned NumArgsToCopy = CI->getNumOperands()-1; - if (CI->getNumOperands()-1 != ParamTys.size() && - !(CI->getNumOperands()-1 > ParamTys.size() && + if (NumArgsToCopy != ParamTys.size() && + !(NumArgsToCopy > ParamTys.size() && Dest->getFunctionType()->isVarArg())) { std::cerr << "WARNING: Call arguments do not match expected number of" << " parameters.\n"; @@ -59,6 +59,8 @@ static void ConvertCallTo(CallInst *CI, Function *Dest) { std::cerr << "Function resolved to: "; WriteAsOperand(std::cerr, Dest); std::cerr << "\n"; + if (NumArgsToCopy > ParamTys.size()) + NumArgsToCopy = ParamTys.size(); } std::vector Params; -- 2.34.1