================
@@ -909,10 +909,12 @@ static bool areExprsSameMacroOrLiteral(const
BinaryOperator *BinOp,
// Left is macro so right macro too
if (Rsr.getBegin().isMacroID()) {
// Both sides are macros so they are same macro or literal
- const llvm::StringRef L = Lexer::getSourceText(
- CharSourceRange::getTokenRange(Lsr), SM, Context->getLangOpts(), 0);
- const llvm::StringRef R = Lexer::getSourceText(
- CharSourceRange::getTokenRange(Rsr), SM, Context->getLangOpts(), 0);
+ const llvm::StringRef L =
+ Lexer::getSourceText(CharSourceRange::getTokenRange(Lsr), SM,
+ Context->getLangOpts(), nullptr);
+ const llvm::StringRef R =
+ Lexer::getSourceText(CharSourceRange::getTokenRange(Rsr), SM,
+ Context->getLangOpts(), nullptr);
----------------
nicovank wrote:
```suggestion
const llvm::StringRef L = Lexer::getSourceText(
CharSourceRange::getTokenRange(Lsr), SM, Context->getLangOpts());
const llvm::StringRef R = Lexer::getSourceText(
CharSourceRange::getTokenRange(Rsr), SM, Context->getLangOpts());
```
The last argument already has a default value of `nullptr`.
https://github.com/llvm/llvm-project/pull/147793
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
