diff options
Diffstat (limited to 'clang-tidy/abseil/DurationAdditionCheck.cpp')
-rw-r--r-- | clang-tidy/abseil/DurationAdditionCheck.cpp | 73 |
1 files changed, 73 insertions, 0 deletions
diff --git a/clang-tidy/abseil/DurationAdditionCheck.cpp b/clang-tidy/abseil/DurationAdditionCheck.cpp new file mode 100644 index 00000000..60047096 --- /dev/null +++ b/clang-tidy/abseil/DurationAdditionCheck.cpp @@ -0,0 +1,73 @@ +//===--- DurationAdditionCheck.cpp - clang-tidy----------------------------===// +// +// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. +// See https://llvm.org/LICENSE.txt for license information. +// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception +// +//===----------------------------------------------------------------------===// + +#include "DurationAdditionCheck.h" +#include "DurationRewriter.h" +#include "clang/AST/ASTContext.h" +#include "clang/ASTMatchers/ASTMatchFinder.h" +#include "clang/Tooling/FixIt.h" + +using namespace clang::ast_matchers; + +namespace clang { +namespace tidy { +namespace abseil { + +void DurationAdditionCheck::registerMatchers(MatchFinder *Finder) { + Finder->addMatcher( + binaryOperator(hasOperatorName("+"), + hasEitherOperand(expr(ignoringParenImpCasts( + callExpr(callee(functionDecl(TimeConversionFunction()) + .bind("function_decl"))) + .bind("call"))))) + .bind("binop"), + this); +} + +void DurationAdditionCheck::check(const MatchFinder::MatchResult &Result) { + const BinaryOperator *Binop = + Result.Nodes.getNodeAs<clang::BinaryOperator>("binop"); + const CallExpr *Call = Result.Nodes.getNodeAs<clang::CallExpr>("call"); + + // Don't try to replace things inside of macro definitions. + if (Binop->getExprLoc().isMacroID() || Binop->getExprLoc().isInvalid()) + return; + + llvm::Optional<DurationScale> Scale = getScaleForTimeInverse( + Result.Nodes.getNodeAs<clang::FunctionDecl>("function_decl")->getName()); + if (!Scale) + return; + + llvm::StringRef TimeFactory = getTimeInverseForScale(*Scale); + + FixItHint Hint; + if (Call == Binop->getLHS()->IgnoreParenImpCasts()) { + Hint = FixItHint::CreateReplacement( + Binop->getSourceRange(), + (llvm::Twine(TimeFactory) + "(" + + tooling::fixit::getText(*Call->getArg(0), *Result.Context) + " + " + + rewriteExprFromNumberToDuration(Result, *Scale, Binop->getRHS()) + ")") + .str()); + } else { + assert(Call == Binop->getRHS()->IgnoreParenImpCasts() && + "Call should be found on the RHS"); + Hint = FixItHint::CreateReplacement( + Binop->getSourceRange(), + (llvm::Twine(TimeFactory) + "(" + + rewriteExprFromNumberToDuration(Result, *Scale, Binop->getLHS()) + + " + " + tooling::fixit::getText(*Call->getArg(0), *Result.Context) + + ")") + .str()); + } + + diag(Binop->getBeginLoc(), "perform addition in the duration domain") << Hint; +} + +} // namespace abseil +} // namespace tidy +} // namespace clang |