From: Chris Lattner <sabre@nondot.org>
Date: Sat, 11 Apr 2009 19:42:43 +0000 (+0000)
Subject: Set proper location info on objc method arguments.
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=7db638d1222bfb2517ac54388e83169a4c76cf7e;p=clang

Set proper location info on objc method arguments.


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

diff --git a/lib/Sema/SemaDeclObjC.cpp b/lib/Sema/SemaDeclObjC.cpp
index 142a5305d4..52b6db59f3 100644
--- a/lib/Sema/SemaDeclObjC.cpp
+++ b/lib/Sema/SemaDeclObjC.cpp
@@ -1431,8 +1431,7 @@ Sema::DeclPtrTy Sema::ActOnMethodDeclaration(
   
   llvm::SmallVector<ParmVarDecl*, 16> Params;
   
-  for (unsigned i = 0; i < Sel.getNumArgs(); i++) {
-    // FIXME: arg->AttrList must be stored too!
+  for (unsigned i = 0, e = Sel.getNumArgs(); i != e; ++i) {
     QualType ArgType, UnpromotedArgType;
     
     if (ArgInfo[i].Type == 0) {
@@ -1445,13 +1444,12 @@ Sema::DeclPtrTy Sema::ActOnMethodDeclaration(
     
     ParmVarDecl* Param;
     if (ArgType == UnpromotedArgType)
-      Param = ParmVarDecl::Create(Context, ObjCMethod,
-                                  SourceLocation(/*FIXME*/),
+      Param = ParmVarDecl::Create(Context, ObjCMethod, ArgInfo[i].NameLoc,
                                   ArgInfo[i].Name, ArgType,
                                   VarDecl::None, 0);
     else
       Param = OriginalParmVarDecl::Create(Context, ObjCMethod,
-                                          SourceLocation(/*FIXME*/),
+                                          ArgInfo[i].NameLoc,
                                           ArgInfo[i].Name, ArgType,
                                           UnpromotedArgType,
                                           VarDecl::None, 0);