Robustify objc method type description (subroutine type) by walking parameters directly.

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

diff --git a/lib/CodeGen/CGDebugInfo.cpp b/lib/CodeGen/CGDebugInfo.cpp
index 5ebf133303efa735283ff4bdf6c651c1e690209e..cc0dbd6bb5e9ce2b654f130386f9947ab65b1089 100644 (file)
--- a/lib/CodeGen/CGDebugInfo.cpp
+++ b/lib/CodeGen/CGDebugInfo.cpp
@@ -1628,25 +1628,20 @@ llvm::DIType CGDebugInfo::getOrCreateFunctionType(const Decl * D, QualType FnTyp
   if (const CXXMethodDecl *Method = dyn_cast<CXXMethodDecl>(D))
     return getOrCreateMethodType(Method, F);
   else if (const ObjCMethodDecl *OMethod = dyn_cast<ObjCMethodDecl>(D)) {
-
-    llvm::DIType MTy = getOrCreateType(FnType, F);
-    llvm::DIArray Args = llvm::DICompositeType(MTy).getTypeArray();
-    assert (Args.getNumElements() && "Invalid number of arguments!");
-
     // Add "self" and "_cmd"
     llvm::SmallVector<llvm::Value *, 16> Elts;
 
     // First element is always return type. For 'void' functions it is NULL.
-    Elts.push_back(Args.getElement(0));
-
+    Elts.push_back(getOrCreateType(OMethod->getResultType(), F));
     // "self" pointer is always first argument.
     Elts.push_back(getOrCreateType(OMethod->getSelfDecl()->getType(), F));
     // "cmd" pointer is always second argument.
     Elts.push_back(getOrCreateType(OMethod->getCmdDecl()->getType(), F));
-    
-    // Copy rest of the arguments.
-    for (unsigned i = 1, e = Args.getNumElements(); i != e; ++i)
-      Elts.push_back(Args.getElement(i));
+    // Get rest of the arguments.
+    for (ObjCMethodDecl::param_iterator PI = OMethod->param_begin(), 
+           PE = OMethod->param_end(); PI != PE; ++PI)
+      Elts.push_back(getOrCreateType((*PI)->getType(), F));
+
     llvm::DIArray EltTypeArray = DBuilder.getOrCreateArray(Elts);
     return DBuilder.createSubroutineType(F, EltTypeArray);
   }