diff options
author | Alp Toker <alp@nuanti.com> | 2014-01-20 20:26:09 +0000 |
---|---|---|
committer | Alp Toker <alp@nuanti.com> | 2014-01-20 20:26:09 +0000 |
commit | c50bf3d128af641f9249f7d1dc37e42b9fd31b81 (patch) | |
tree | 82cb9d4ca1cc715bbf62f39de4b4e27d8b705beb /lib/AST/Mangle.cpp | |
parent | 55c1c8ac7914747a84329aab72200822b8e8b5c2 (diff) | |
download | clang-c50bf3d128af641f9249f7d1dc37e42b9fd31b81.tar.gz |
Rename FunctionProtoType accessors from 'arguments' to 'parameters'
Fix a perennial source of confusion in the clang type system: Declarations and
function prototypes have parameters to which arguments are supplied, so calling
these 'arguments' was a stretch even in C mode, let alone C++ where default
arguments, templates and overloading make the distinction important to get
right.
Readability win across the board, especially in the casting, ADL and
overloading implementations which make a lot more sense at a glance now.
Will keep an eye on the builders and update dependent projects shortly.
No functional change.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@199686 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/AST/Mangle.cpp')
-rw-r--r-- | lib/AST/Mangle.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/AST/Mangle.cpp b/lib/AST/Mangle.cpp index ab2434415b..6ab2406079 100644 --- a/lib/AST/Mangle.cpp +++ b/lib/AST/Mangle.cpp @@ -163,8 +163,8 @@ void MangleContext::mangleName(const NamedDecl *D, raw_ostream &Out) { if (const CXXMethodDecl *MD = dyn_cast<CXXMethodDecl>(FD)) if (!MD->isStatic()) ++ArgWords; - for (FunctionProtoType::arg_type_iterator Arg = Proto->arg_type_begin(), - ArgEnd = Proto->arg_type_end(); + for (FunctionProtoType::param_type_iterator Arg = Proto->param_type_begin(), + ArgEnd = Proto->param_type_end(); Arg != ArgEnd; ++Arg) { QualType AT = *Arg; // Size should be aligned to DWORD boundary |