From: Chris Lattner <sabre@nondot.org>
Date: Wed, 29 Aug 2007 06:38:44 +0000 (+0000)
Subject: remove some ugly code now that implicit defs are being generated in this case, thanks... 
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=1d40060a3794acf967240fdbf1b39e4f37f90dc9;p=clang

remove some ugly code now that implicit defs are being generated in this case, thanks Steve!


git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@41560 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/CodeGen/CGExpr.cpp b/CodeGen/CGExpr.cpp
index a4b168d717..2981ed609a 100644
--- a/CodeGen/CGExpr.cpp
+++ b/CodeGen/CGExpr.cpp
@@ -407,18 +407,6 @@ RValue CodeGenFunction::EmitCallExpr(const CallExpr *E) {
     if (!hasAggregateLLVMType(ArgTy)) {
       // Scalar argument is passed by-value.
       Args.push_back(EmitScalarExpr(E->getArg(i)));
-      
-      if (ArgTyIt == ArgTyEnd) {
-        // Otherwise, if passing through "..." or to a function with no prototype,
-        // perform the "default argument promotions" (C99 6.5.2.2p6), which
-        // includes the usual unary conversions, but also promotes float to
-        // double.
-        // FIXME: remove this when the impcast is in place.
-        if (Args.back()->getType() == llvm::Type::FloatTy)
-          Args.back() = Builder.CreateFPExt(Args.back(), llvm::Type::DoubleTy,
-                                            "tmp");
-        // FIXME: Remove ArgIt when this is gone.
-      }
     } else if (ArgTy->isComplexType()) {
       // Make a temporary alloca to pass the argument.
       llvm::Value *DestMem = CreateTempAlloca(ConvertType(ArgTy));
@@ -429,9 +417,6 @@ RValue CodeGenFunction::EmitCallExpr(const CallExpr *E) {
       EmitAggExpr(E->getArg(i), DestMem, false);
       Args.push_back(DestMem);
     }
-    
-    if (ArgTyIt != ArgTyEnd)
-      ++ArgTyIt;
   }
   
   llvm::Value *V = Builder.CreateCall(Callee, &Args[0], &Args[0]+Args.size());