-
Notifications
You must be signed in to change notification settings - Fork 0
Expand file tree
/
Copy pathPerlUndefSetpvXCheck.cpp
More file actions
54 lines (46 loc) · 1.74 KB
/
PerlUndefSetpvXCheck.cpp
File metadata and controls
54 lines (46 loc) · 1.74 KB
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
#include "PerlCheck.h"
#include "PerlUndefSetpvXCheck.h"
#include "llvm/Support/FormatVariadic.h"
#include <cassert>
#include <iostream>
using namespace clang;
using namespace clang::tidy;
using namespace clang::ast_matchers;
using namespace perl_check;
PerlUndefSetpvXCheck::PerlUndefSetpvXCheck(
StringRef Name, ClangTidyContext* Context,
StringRef PvMacro_,
StringRef PvCallTo_
) :
ClangTidyCheck(Name, Context),
PvMacro(PvMacro_),
PvCallTo(PvCallTo_),
UseMultiplicity(Options.getLocalOrGlobal("PerlCheckMultiplicity", 0))
{
}
void PerlUndefSetpvXCheck::registerMatchers(MatchFinder* Finder)
{
Finder->addMatcher(
callExpr(
callee(functionDecl(hasName(PvCallTo))),
isExpandedFromMacro(PvMacro),
hasArgument(0+UseMultiplicity, expr().bind("sv")),
hasArgument(1+UseMultiplicity, expr().bind("pv"))
).bind("call"), this);
}
void PerlUndefSetpvXCheck::check(const MatchFinder::MatchResult& Result)
{
const auto *matchedCall = Result.Nodes.getNodeAs<CallExpr>("call");
const auto *svExpr = Result.Nodes.getNodeAs<Expr>("sv");
const auto *pvExpr = Result.Nodes.getNodeAs<Expr>("pv");
const LangOptions &Opts = getLangOpts();
auto argString = getArgText(matchedCall, Result, Opts, UseMultiplicity);
if (!pvExpr->isConstantInitializer(*Result.Context, false) ||
!pvExpr->isNullPointerConstant(
*Result.Context, Expr::NPC_NeverValueDependent)) {
return;
}
std::string repl = llvm::formatv("sv_set_undef({0})", argString(0));
auto hint = FixItHint::CreateReplacement(matchedCall->getSourceRange(), repl);
diag(matchedCall->getExprLoc(), "%0(..., NULL) better written as sv_set_undef()") << PvMacro << hint;
}