Fix a bug that cause a crash resolving questionable function calls
authorChris Lattner <sabre@nondot.org>
Fri, 14 Feb 2003 19:12:29 +0000 (19:12 +0000)
committerChris Lattner <sabre@nondot.org>
Fri, 14 Feb 2003 19:12:29 +0000 (19:12 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@5567 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Transforms/IPO/FunctionResolution.cpp

index 9c5580b3b26ff2809da1a345c615faf27fdbe328..3d5991aba988864e89fdc9862d8699b6a105ee3d 100644 (file)
@@ -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<Value*> Params;