diff options
author | Faisal Vali <faisalv@yahoo.com> | 2017-09-30 13:58:38 +0000 |
---|---|---|
committer | Faisal Vali <faisalv@yahoo.com> | 2017-09-30 13:58:38 +0000 |
commit | 333133e2c8c08b78f8bff5610c387040034c0168 (patch) | |
tree | c1aaf23b47d5df69c4e07e01a32dc424683311d3 /clang/lib | |
parent | 70947e222471393f034c552ff8c08d301e75e534 (diff) | |
download | bcm5719-llvm-333133e2c8c08b78f8bff5610c387040034c0168.tar.gz bcm5719-llvm-333133e2c8c08b78f8bff5610c387040034c0168.zip |
[NFC] Remove superfluous parameter
- MacroArgs already knows the maximum number of arguments that can be supplied to the macro. No need to pass MacroInfo (information about the macro definition) to the call to getPreExpArgument (which by the way might benefit from being called getExpandedArgument() ?) for it to compute the number of arguments.
llvm-svn: 314593
Diffstat (limited to 'clang/lib')
-rw-r--r-- | clang/lib/Lex/MacroArgs.cpp | 11 | ||||
-rw-r--r-- | clang/lib/Lex/TokenLexer.cpp | 2 |
2 files changed, 6 insertions, 7 deletions
diff --git a/clang/lib/Lex/MacroArgs.cpp b/clang/lib/Lex/MacroArgs.cpp index 37a7d5c1fff..de54f392a48 100644 --- a/clang/lib/Lex/MacroArgs.cpp +++ b/clang/lib/Lex/MacroArgs.cpp @@ -150,14 +150,13 @@ bool MacroArgs::ArgNeedsPreexpansion(const Token *ArgTok, /// getPreExpArgument - Return the pre-expanded form of the specified /// argument. -const std::vector<Token> & -MacroArgs::getPreExpArgument(unsigned Arg, const MacroInfo *MI, - Preprocessor &PP) { - assert(Arg < MI->getNumParams() && "Invalid argument number!"); +const std::vector<Token> &MacroArgs::getPreExpArgument(unsigned Arg, + Preprocessor &PP) { + assert(Arg < getNumMacroArguments() && "Invalid argument number!"); // If we have already computed this, return it. - if (PreExpArgTokens.size() < MI->getNumParams()) - PreExpArgTokens.resize(MI->getNumParams()); + if (PreExpArgTokens.size() < getNumMacroArguments()) + PreExpArgTokens.resize(getNumMacroArguments()); std::vector<Token> &Result = PreExpArgTokens[Arg]; if (!Result.empty()) return Result; diff --git a/clang/lib/Lex/TokenLexer.cpp b/clang/lib/Lex/TokenLexer.cpp index 10e21b3b048..4341d56b4e3 100644 --- a/clang/lib/Lex/TokenLexer.cpp +++ b/clang/lib/Lex/TokenLexer.cpp @@ -275,7 +275,7 @@ void TokenLexer::ExpandFunctionArguments() { // avoids some work in common cases. const Token *ArgTok = ActualArgs->getUnexpArgument(ArgNo); if (ActualArgs->ArgNeedsPreexpansion(ArgTok, PP)) - ResultArgToks = &ActualArgs->getPreExpArgument(ArgNo, Macro, PP)[0]; + ResultArgToks = &ActualArgs->getPreExpArgument(ArgNo, PP)[0]; else ResultArgToks = ArgTok; // Use non-preexpanded tokens. |