From 9b75edbd14cbd0604b22162f411d2d52ef3d0b6b Mon Sep 17 00:00:00 2001
From: Torok Edwin <edwintorok@gmail.com>
Date: Fri, 22 May 2009 07:12:05 +0000
Subject: [PATCH] Revert this. There's no way to verifiy indirect calls, and an
 optimizer can turn indirect call into direct call, thus the verifier would
 reject something it previously accepted.

llvm-svn: 72249
---
 lib/VMCore/Verifier.cpp         | 25 -------------------------
 test/Assembler/2009-05-22-CC.ll |  9 ---------
 test/Verifier/2009-05-22-CC.ll  | 11 -----------
 3 files changed, 45 deletions(-)
 delete mode 100644 test/Assembler/2009-05-22-CC.ll
 delete mode 100644 test/Verifier/2009-05-22-CC.ll

diff --git a/lib/VMCore/Verifier.cpp b/lib/VMCore/Verifier.cpp
index 65d0a8236b9..fc4cfcfe45d 100644
--- a/lib/VMCore/Verifier.cpp
+++ b/lib/VMCore/Verifier.cpp
@@ -1024,31 +1024,6 @@ void Verifier::VerifyCallSite(CallSite CS) {
             "Call parameter type does not match function signature!",
             CS.getArgument(i), FTy->getParamType(i), I);
 
-  Assert2(CS.getType() == FTy->getReturnType(),
-          "Call return type does not match function signature!",
-          CS.getInstruction(), FTy->getReturnType());
-
-  // Verify calling convention for direct calls
-  Value *CalledF = CS.getCalledValue()->stripPointerCasts();
-  if (Function *F = dyn_cast<Function>(CalledF)) {
-    unsigned CC1 = CS.getCallingConv();
-    unsigned CC2 = F->getCallingConv();
-    if(CC1 != CC2) {
-      // tolerate some mismatch among C prototype and LLVM-specific calling conv
-      if (CC2 >= CallingConv::FirstTargetCC || 
-          CC1 >= CallingConv::FirstTargetCC) {
-        Instruction *I = CS.getInstruction()->clone();
-        if (CallInst *CI = dyn_cast<CallInst>(I)) {
-          CI->setCallingConv(F->getCallingConv());
-        } else
-          cast<InvokeInst>(I)->setCallingConv(F->getCallingConv());
-        Assert2(0,"Calling convention does not match function signature!",
-                CS.getInstruction(), I);
-        delete I;
-      }
-    }
-  }
-
   if (CS.getCalledValue()->getNameLen() < 5 ||
       strncmp(CS.getCalledValue()->getNameStart(), "llvm.", 5) != 0) {
     // Verify that none of the arguments are metadata...
diff --git a/test/Assembler/2009-05-22-CC.ll b/test/Assembler/2009-05-22-CC.ll
deleted file mode 100644
index ba80d64ac28..00000000000
--- a/test/Assembler/2009-05-22-CC.ll
+++ /dev/null
@@ -1,9 +0,0 @@
-; RUN: llvm-as < %s
-
-; Verify that calls with correct calling conv are accepted
-declare x86_stdcallcc i32 @re_string_construct(i8* inreg %pstr, i8* inreg %str, i32 inreg %len, i8* %trans, i32 %icase, i8* %dfa);
-define void @main() {
-entry:
-	%0 = call x86_stdcallcc i32 (...)* bitcast (i32 (i8*, i8*, i32, i8*, i32, i8*)* @re_string_construct to i32 (...)*)(i32 inreg 0, i32 inreg 0, i32 inreg 0, i32 0, i32 0, i8* inttoptr (i32 673194176 to i8*));
-        ret void
-}
diff --git a/test/Verifier/2009-05-22-CC.ll b/test/Verifier/2009-05-22-CC.ll
deleted file mode 100644
index 7775dede03a..00000000000
--- a/test/Verifier/2009-05-22-CC.ll
+++ /dev/null
@@ -1,11 +0,0 @@
-; RUN: not llvm-as < %s |& grep {Calling convention does not match function signature}
-; PR 4239
-
-; Verify that the calling convention on the call instruction matches the
-; declared calling convention
-declare x86_stdcallcc i32 @re_string_construct(i8* inreg %pstr, i8* inreg %str, i32 inreg %len, i8* %trans, i32 %icase, i8* %dfa);
-define void @main() {
-entry:
-	%0 = call i32 (...)* bitcast (i32 (i8*, i8*, i32, i8*, i32, i8*)* @re_string_construct to i32 (...)*)(i32 inreg 0, i32 inreg 0, i32 inreg 0, i32 0, i32 0, i8* inttoptr (i32 673194176 to i8*))
-        ret void
-}