From 582ae9916ae565811b9ca7d8e6ebe91e7cdb442a Mon Sep 17 00:00:00 2001 From: Argyrios Kyrtzidis Date: Sat, 30 Jul 2011 17:23:28 +0000 Subject: Not sure why we bother updating FunctionDecl's EndRangeLoc in FunctionDecl::setParams. EndRangeLoc should always be set to at least the ending paren or brace. llvm-svn: 136573 --- clang/lib/AST/Decl.cpp | 5 ----- 1 file changed, 5 deletions(-) (limited to 'clang') diff --git a/clang/lib/AST/Decl.cpp b/clang/lib/AST/Decl.cpp index 64fd355446a..32b9a624e0e 100644 --- a/clang/lib/AST/Decl.cpp +++ b/clang/lib/AST/Decl.cpp @@ -1697,11 +1697,6 @@ void FunctionDecl::setParams(ASTContext &C, void *Mem = C.Allocate(sizeof(ParmVarDecl*)*NumParams); ParamInfo = new (Mem) ParmVarDecl*[NumParams]; memcpy(ParamInfo, NewParamInfo, sizeof(ParmVarDecl*)*NumParams); - - // Update source range. The check below allows us to set EndRangeLoc before - // setting the parameters. - if (EndRangeLoc.isInvalid() || EndRangeLoc == getLocation()) - EndRangeLoc = NewParamInfo[NumParams-1]->getLocEnd(); } } -- cgit v1.2.3