1e5dd7070Spatrick //===- Consumed.cpp -------------------------------------------------------===//
2e5dd7070Spatrick //
3e5dd7070Spatrick // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
4e5dd7070Spatrick // See https://llvm.org/LICENSE.txt for license information.
5e5dd7070Spatrick // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
6e5dd7070Spatrick //
7e5dd7070Spatrick //===----------------------------------------------------------------------===//
8e5dd7070Spatrick //
9e5dd7070Spatrick // A intra-procedural analysis for checking consumed properties. This is based,
10e5dd7070Spatrick // in part, on research on linear types.
11e5dd7070Spatrick //
12e5dd7070Spatrick //===----------------------------------------------------------------------===//
13e5dd7070Spatrick
14e5dd7070Spatrick #include "clang/Analysis/Analyses/Consumed.h"
15e5dd7070Spatrick #include "clang/AST/Attr.h"
16e5dd7070Spatrick #include "clang/AST/Decl.h"
17e5dd7070Spatrick #include "clang/AST/DeclCXX.h"
18e5dd7070Spatrick #include "clang/AST/Expr.h"
19e5dd7070Spatrick #include "clang/AST/ExprCXX.h"
20e5dd7070Spatrick #include "clang/AST/Stmt.h"
21e5dd7070Spatrick #include "clang/AST/StmtVisitor.h"
22e5dd7070Spatrick #include "clang/AST/Type.h"
23e5dd7070Spatrick #include "clang/Analysis/Analyses/PostOrderCFGView.h"
24e5dd7070Spatrick #include "clang/Analysis/AnalysisDeclContext.h"
25e5dd7070Spatrick #include "clang/Analysis/CFG.h"
26e5dd7070Spatrick #include "clang/Basic/LLVM.h"
27e5dd7070Spatrick #include "clang/Basic/OperatorKinds.h"
28e5dd7070Spatrick #include "clang/Basic/SourceLocation.h"
29e5dd7070Spatrick #include "llvm/ADT/DenseMap.h"
30e5dd7070Spatrick #include "llvm/ADT/STLExtras.h"
31e5dd7070Spatrick #include "llvm/ADT/StringRef.h"
32e5dd7070Spatrick #include "llvm/Support/Casting.h"
33e5dd7070Spatrick #include "llvm/Support/ErrorHandling.h"
34e5dd7070Spatrick #include <cassert>
35e5dd7070Spatrick #include <memory>
36*12c85518Srobert #include <optional>
37e5dd7070Spatrick #include <utility>
38e5dd7070Spatrick
39e5dd7070Spatrick // TODO: Adjust states of args to constructors in the same way that arguments to
40e5dd7070Spatrick // function calls are handled.
41e5dd7070Spatrick // TODO: Use information from tests in for- and while-loop conditional.
42e5dd7070Spatrick // TODO: Add notes about the actual and expected state for
43e5dd7070Spatrick // TODO: Correctly identify unreachable blocks when chaining boolean operators.
44e5dd7070Spatrick // TODO: Adjust the parser and AttributesList class to support lists of
45e5dd7070Spatrick // identifiers.
46e5dd7070Spatrick // TODO: Warn about unreachable code.
47e5dd7070Spatrick // TODO: Switch to using a bitmap to track unreachable blocks.
48e5dd7070Spatrick // TODO: Handle variable definitions, e.g. bool valid = x.isValid();
49e5dd7070Spatrick // if (valid) ...; (Deferred)
50e5dd7070Spatrick // TODO: Take notes on state transitions to provide better warning messages.
51e5dd7070Spatrick // (Deferred)
52e5dd7070Spatrick // TODO: Test nested conditionals: A) Checking the same value multiple times,
53e5dd7070Spatrick // and 2) Checking different values. (Deferred)
54e5dd7070Spatrick
55e5dd7070Spatrick using namespace clang;
56e5dd7070Spatrick using namespace consumed;
57e5dd7070Spatrick
58e5dd7070Spatrick // Key method definition
59e5dd7070Spatrick ConsumedWarningsHandlerBase::~ConsumedWarningsHandlerBase() = default;
60e5dd7070Spatrick
getFirstStmtLoc(const CFGBlock * Block)61e5dd7070Spatrick static SourceLocation getFirstStmtLoc(const CFGBlock *Block) {
62e5dd7070Spatrick // Find the source location of the first statement in the block, if the block
63e5dd7070Spatrick // is not empty.
64e5dd7070Spatrick for (const auto &B : *Block)
65*12c85518Srobert if (std::optional<CFGStmt> CS = B.getAs<CFGStmt>())
66e5dd7070Spatrick return CS->getStmt()->getBeginLoc();
67e5dd7070Spatrick
68e5dd7070Spatrick // Block is empty.
69e5dd7070Spatrick // If we have one successor, return the first statement in that block
70e5dd7070Spatrick if (Block->succ_size() == 1 && *Block->succ_begin())
71e5dd7070Spatrick return getFirstStmtLoc(*Block->succ_begin());
72e5dd7070Spatrick
73e5dd7070Spatrick return {};
74e5dd7070Spatrick }
75e5dd7070Spatrick
getLastStmtLoc(const CFGBlock * Block)76e5dd7070Spatrick static SourceLocation getLastStmtLoc(const CFGBlock *Block) {
77e5dd7070Spatrick // Find the source location of the last statement in the block, if the block
78e5dd7070Spatrick // is not empty.
79e5dd7070Spatrick if (const Stmt *StmtNode = Block->getTerminatorStmt()) {
80e5dd7070Spatrick return StmtNode->getBeginLoc();
81e5dd7070Spatrick } else {
82e5dd7070Spatrick for (CFGBlock::const_reverse_iterator BI = Block->rbegin(),
83e5dd7070Spatrick BE = Block->rend(); BI != BE; ++BI) {
84*12c85518Srobert if (std::optional<CFGStmt> CS = BI->getAs<CFGStmt>())
85e5dd7070Spatrick return CS->getStmt()->getBeginLoc();
86e5dd7070Spatrick }
87e5dd7070Spatrick }
88e5dd7070Spatrick
89e5dd7070Spatrick // If we have one successor, return the first statement in that block
90e5dd7070Spatrick SourceLocation Loc;
91e5dd7070Spatrick if (Block->succ_size() == 1 && *Block->succ_begin())
92e5dd7070Spatrick Loc = getFirstStmtLoc(*Block->succ_begin());
93e5dd7070Spatrick if (Loc.isValid())
94e5dd7070Spatrick return Loc;
95e5dd7070Spatrick
96e5dd7070Spatrick // If we have one predecessor, return the last statement in that block
97e5dd7070Spatrick if (Block->pred_size() == 1 && *Block->pred_begin())
98e5dd7070Spatrick return getLastStmtLoc(*Block->pred_begin());
99e5dd7070Spatrick
100e5dd7070Spatrick return Loc;
101e5dd7070Spatrick }
102e5dd7070Spatrick
invertConsumedUnconsumed(ConsumedState State)103e5dd7070Spatrick static ConsumedState invertConsumedUnconsumed(ConsumedState State) {
104e5dd7070Spatrick switch (State) {
105e5dd7070Spatrick case CS_Unconsumed:
106e5dd7070Spatrick return CS_Consumed;
107e5dd7070Spatrick case CS_Consumed:
108e5dd7070Spatrick return CS_Unconsumed;
109e5dd7070Spatrick case CS_None:
110e5dd7070Spatrick return CS_None;
111e5dd7070Spatrick case CS_Unknown:
112e5dd7070Spatrick return CS_Unknown;
113e5dd7070Spatrick }
114e5dd7070Spatrick llvm_unreachable("invalid enum");
115e5dd7070Spatrick }
116e5dd7070Spatrick
isCallableInState(const CallableWhenAttr * CWAttr,ConsumedState State)117e5dd7070Spatrick static bool isCallableInState(const CallableWhenAttr *CWAttr,
118e5dd7070Spatrick ConsumedState State) {
119e5dd7070Spatrick for (const auto &S : CWAttr->callableStates()) {
120e5dd7070Spatrick ConsumedState MappedAttrState = CS_None;
121e5dd7070Spatrick
122e5dd7070Spatrick switch (S) {
123e5dd7070Spatrick case CallableWhenAttr::Unknown:
124e5dd7070Spatrick MappedAttrState = CS_Unknown;
125e5dd7070Spatrick break;
126e5dd7070Spatrick
127e5dd7070Spatrick case CallableWhenAttr::Unconsumed:
128e5dd7070Spatrick MappedAttrState = CS_Unconsumed;
129e5dd7070Spatrick break;
130e5dd7070Spatrick
131e5dd7070Spatrick case CallableWhenAttr::Consumed:
132e5dd7070Spatrick MappedAttrState = CS_Consumed;
133e5dd7070Spatrick break;
134e5dd7070Spatrick }
135e5dd7070Spatrick
136e5dd7070Spatrick if (MappedAttrState == State)
137e5dd7070Spatrick return true;
138e5dd7070Spatrick }
139e5dd7070Spatrick
140e5dd7070Spatrick return false;
141e5dd7070Spatrick }
142e5dd7070Spatrick
isConsumableType(const QualType & QT)143e5dd7070Spatrick static bool isConsumableType(const QualType &QT) {
144e5dd7070Spatrick if (QT->isPointerType() || QT->isReferenceType())
145e5dd7070Spatrick return false;
146e5dd7070Spatrick
147e5dd7070Spatrick if (const CXXRecordDecl *RD = QT->getAsCXXRecordDecl())
148e5dd7070Spatrick return RD->hasAttr<ConsumableAttr>();
149e5dd7070Spatrick
150e5dd7070Spatrick return false;
151e5dd7070Spatrick }
152e5dd7070Spatrick
isAutoCastType(const QualType & QT)153e5dd7070Spatrick static bool isAutoCastType(const QualType &QT) {
154e5dd7070Spatrick if (QT->isPointerType() || QT->isReferenceType())
155e5dd7070Spatrick return false;
156e5dd7070Spatrick
157e5dd7070Spatrick if (const CXXRecordDecl *RD = QT->getAsCXXRecordDecl())
158e5dd7070Spatrick return RD->hasAttr<ConsumableAutoCastAttr>();
159e5dd7070Spatrick
160e5dd7070Spatrick return false;
161e5dd7070Spatrick }
162e5dd7070Spatrick
isSetOnReadPtrType(const QualType & QT)163e5dd7070Spatrick static bool isSetOnReadPtrType(const QualType &QT) {
164e5dd7070Spatrick if (const CXXRecordDecl *RD = QT->getPointeeCXXRecordDecl())
165e5dd7070Spatrick return RD->hasAttr<ConsumableSetOnReadAttr>();
166e5dd7070Spatrick return false;
167e5dd7070Spatrick }
168e5dd7070Spatrick
isKnownState(ConsumedState State)169e5dd7070Spatrick static bool isKnownState(ConsumedState State) {
170e5dd7070Spatrick switch (State) {
171e5dd7070Spatrick case CS_Unconsumed:
172e5dd7070Spatrick case CS_Consumed:
173e5dd7070Spatrick return true;
174e5dd7070Spatrick case CS_None:
175e5dd7070Spatrick case CS_Unknown:
176e5dd7070Spatrick return false;
177e5dd7070Spatrick }
178e5dd7070Spatrick llvm_unreachable("invalid enum");
179e5dd7070Spatrick }
180e5dd7070Spatrick
isRValueRef(QualType ParamType)181e5dd7070Spatrick static bool isRValueRef(QualType ParamType) {
182e5dd7070Spatrick return ParamType->isRValueReferenceType();
183e5dd7070Spatrick }
184e5dd7070Spatrick
isTestingFunction(const FunctionDecl * FunDecl)185e5dd7070Spatrick static bool isTestingFunction(const FunctionDecl *FunDecl) {
186e5dd7070Spatrick return FunDecl->hasAttr<TestTypestateAttr>();
187e5dd7070Spatrick }
188e5dd7070Spatrick
isPointerOrRef(QualType ParamType)189e5dd7070Spatrick static bool isPointerOrRef(QualType ParamType) {
190e5dd7070Spatrick return ParamType->isPointerType() || ParamType->isReferenceType();
191e5dd7070Spatrick }
192e5dd7070Spatrick
mapConsumableAttrState(const QualType QT)193e5dd7070Spatrick static ConsumedState mapConsumableAttrState(const QualType QT) {
194e5dd7070Spatrick assert(isConsumableType(QT));
195e5dd7070Spatrick
196e5dd7070Spatrick const ConsumableAttr *CAttr =
197e5dd7070Spatrick QT->getAsCXXRecordDecl()->getAttr<ConsumableAttr>();
198e5dd7070Spatrick
199e5dd7070Spatrick switch (CAttr->getDefaultState()) {
200e5dd7070Spatrick case ConsumableAttr::Unknown:
201e5dd7070Spatrick return CS_Unknown;
202e5dd7070Spatrick case ConsumableAttr::Unconsumed:
203e5dd7070Spatrick return CS_Unconsumed;
204e5dd7070Spatrick case ConsumableAttr::Consumed:
205e5dd7070Spatrick return CS_Consumed;
206e5dd7070Spatrick }
207e5dd7070Spatrick llvm_unreachable("invalid enum");
208e5dd7070Spatrick }
209e5dd7070Spatrick
210e5dd7070Spatrick static ConsumedState
mapParamTypestateAttrState(const ParamTypestateAttr * PTAttr)211e5dd7070Spatrick mapParamTypestateAttrState(const ParamTypestateAttr *PTAttr) {
212e5dd7070Spatrick switch (PTAttr->getParamState()) {
213e5dd7070Spatrick case ParamTypestateAttr::Unknown:
214e5dd7070Spatrick return CS_Unknown;
215e5dd7070Spatrick case ParamTypestateAttr::Unconsumed:
216e5dd7070Spatrick return CS_Unconsumed;
217e5dd7070Spatrick case ParamTypestateAttr::Consumed:
218e5dd7070Spatrick return CS_Consumed;
219e5dd7070Spatrick }
220e5dd7070Spatrick llvm_unreachable("invalid_enum");
221e5dd7070Spatrick }
222e5dd7070Spatrick
223e5dd7070Spatrick static ConsumedState
mapReturnTypestateAttrState(const ReturnTypestateAttr * RTSAttr)224e5dd7070Spatrick mapReturnTypestateAttrState(const ReturnTypestateAttr *RTSAttr) {
225e5dd7070Spatrick switch (RTSAttr->getState()) {
226e5dd7070Spatrick case ReturnTypestateAttr::Unknown:
227e5dd7070Spatrick return CS_Unknown;
228e5dd7070Spatrick case ReturnTypestateAttr::Unconsumed:
229e5dd7070Spatrick return CS_Unconsumed;
230e5dd7070Spatrick case ReturnTypestateAttr::Consumed:
231e5dd7070Spatrick return CS_Consumed;
232e5dd7070Spatrick }
233e5dd7070Spatrick llvm_unreachable("invalid enum");
234e5dd7070Spatrick }
235e5dd7070Spatrick
mapSetTypestateAttrState(const SetTypestateAttr * STAttr)236e5dd7070Spatrick static ConsumedState mapSetTypestateAttrState(const SetTypestateAttr *STAttr) {
237e5dd7070Spatrick switch (STAttr->getNewState()) {
238e5dd7070Spatrick case SetTypestateAttr::Unknown:
239e5dd7070Spatrick return CS_Unknown;
240e5dd7070Spatrick case SetTypestateAttr::Unconsumed:
241e5dd7070Spatrick return CS_Unconsumed;
242e5dd7070Spatrick case SetTypestateAttr::Consumed:
243e5dd7070Spatrick return CS_Consumed;
244e5dd7070Spatrick }
245e5dd7070Spatrick llvm_unreachable("invalid_enum");
246e5dd7070Spatrick }
247e5dd7070Spatrick
stateToString(ConsumedState State)248e5dd7070Spatrick static StringRef stateToString(ConsumedState State) {
249e5dd7070Spatrick switch (State) {
250e5dd7070Spatrick case consumed::CS_None:
251e5dd7070Spatrick return "none";
252e5dd7070Spatrick
253e5dd7070Spatrick case consumed::CS_Unknown:
254e5dd7070Spatrick return "unknown";
255e5dd7070Spatrick
256e5dd7070Spatrick case consumed::CS_Unconsumed:
257e5dd7070Spatrick return "unconsumed";
258e5dd7070Spatrick
259e5dd7070Spatrick case consumed::CS_Consumed:
260e5dd7070Spatrick return "consumed";
261e5dd7070Spatrick }
262e5dd7070Spatrick llvm_unreachable("invalid enum");
263e5dd7070Spatrick }
264e5dd7070Spatrick
testsFor(const FunctionDecl * FunDecl)265e5dd7070Spatrick static ConsumedState testsFor(const FunctionDecl *FunDecl) {
266e5dd7070Spatrick assert(isTestingFunction(FunDecl));
267e5dd7070Spatrick switch (FunDecl->getAttr<TestTypestateAttr>()->getTestState()) {
268e5dd7070Spatrick case TestTypestateAttr::Unconsumed:
269e5dd7070Spatrick return CS_Unconsumed;
270e5dd7070Spatrick case TestTypestateAttr::Consumed:
271e5dd7070Spatrick return CS_Consumed;
272e5dd7070Spatrick }
273e5dd7070Spatrick llvm_unreachable("invalid enum");
274e5dd7070Spatrick }
275e5dd7070Spatrick
276e5dd7070Spatrick namespace {
277e5dd7070Spatrick
278e5dd7070Spatrick struct VarTestResult {
279e5dd7070Spatrick const VarDecl *Var;
280e5dd7070Spatrick ConsumedState TestsFor;
281e5dd7070Spatrick };
282e5dd7070Spatrick
283e5dd7070Spatrick } // namespace
284e5dd7070Spatrick
285e5dd7070Spatrick namespace clang {
286e5dd7070Spatrick namespace consumed {
287e5dd7070Spatrick
288e5dd7070Spatrick enum EffectiveOp {
289e5dd7070Spatrick EO_And,
290e5dd7070Spatrick EO_Or
291e5dd7070Spatrick };
292e5dd7070Spatrick
293e5dd7070Spatrick class PropagationInfo {
294e5dd7070Spatrick enum {
295e5dd7070Spatrick IT_None,
296e5dd7070Spatrick IT_State,
297e5dd7070Spatrick IT_VarTest,
298e5dd7070Spatrick IT_BinTest,
299e5dd7070Spatrick IT_Var,
300e5dd7070Spatrick IT_Tmp
301e5dd7070Spatrick } InfoType = IT_None;
302e5dd7070Spatrick
303e5dd7070Spatrick struct BinTestTy {
304e5dd7070Spatrick const BinaryOperator *Source;
305e5dd7070Spatrick EffectiveOp EOp;
306e5dd7070Spatrick VarTestResult LTest;
307e5dd7070Spatrick VarTestResult RTest;
308e5dd7070Spatrick };
309e5dd7070Spatrick
310e5dd7070Spatrick union {
311e5dd7070Spatrick ConsumedState State;
312e5dd7070Spatrick VarTestResult VarTest;
313e5dd7070Spatrick const VarDecl *Var;
314e5dd7070Spatrick const CXXBindTemporaryExpr *Tmp;
315e5dd7070Spatrick BinTestTy BinTest;
316e5dd7070Spatrick };
317e5dd7070Spatrick
318e5dd7070Spatrick public:
319e5dd7070Spatrick PropagationInfo() = default;
PropagationInfo(const VarTestResult & VarTest)320e5dd7070Spatrick PropagationInfo(const VarTestResult &VarTest)
321e5dd7070Spatrick : InfoType(IT_VarTest), VarTest(VarTest) {}
322e5dd7070Spatrick
PropagationInfo(const VarDecl * Var,ConsumedState TestsFor)323e5dd7070Spatrick PropagationInfo(const VarDecl *Var, ConsumedState TestsFor)
324e5dd7070Spatrick : InfoType(IT_VarTest) {
325e5dd7070Spatrick VarTest.Var = Var;
326e5dd7070Spatrick VarTest.TestsFor = TestsFor;
327e5dd7070Spatrick }
328e5dd7070Spatrick
PropagationInfo(const BinaryOperator * Source,EffectiveOp EOp,const VarTestResult & LTest,const VarTestResult & RTest)329e5dd7070Spatrick PropagationInfo(const BinaryOperator *Source, EffectiveOp EOp,
330e5dd7070Spatrick const VarTestResult <est, const VarTestResult &RTest)
331e5dd7070Spatrick : InfoType(IT_BinTest) {
332e5dd7070Spatrick BinTest.Source = Source;
333e5dd7070Spatrick BinTest.EOp = EOp;
334e5dd7070Spatrick BinTest.LTest = LTest;
335e5dd7070Spatrick BinTest.RTest = RTest;
336e5dd7070Spatrick }
337e5dd7070Spatrick
PropagationInfo(const BinaryOperator * Source,EffectiveOp EOp,const VarDecl * LVar,ConsumedState LTestsFor,const VarDecl * RVar,ConsumedState RTestsFor)338e5dd7070Spatrick PropagationInfo(const BinaryOperator *Source, EffectiveOp EOp,
339e5dd7070Spatrick const VarDecl *LVar, ConsumedState LTestsFor,
340e5dd7070Spatrick const VarDecl *RVar, ConsumedState RTestsFor)
341e5dd7070Spatrick : InfoType(IT_BinTest) {
342e5dd7070Spatrick BinTest.Source = Source;
343e5dd7070Spatrick BinTest.EOp = EOp;
344e5dd7070Spatrick BinTest.LTest.Var = LVar;
345e5dd7070Spatrick BinTest.LTest.TestsFor = LTestsFor;
346e5dd7070Spatrick BinTest.RTest.Var = RVar;
347e5dd7070Spatrick BinTest.RTest.TestsFor = RTestsFor;
348e5dd7070Spatrick }
349e5dd7070Spatrick
PropagationInfo(ConsumedState State)350e5dd7070Spatrick PropagationInfo(ConsumedState State)
351e5dd7070Spatrick : InfoType(IT_State), State(State) {}
PropagationInfo(const VarDecl * Var)352e5dd7070Spatrick PropagationInfo(const VarDecl *Var) : InfoType(IT_Var), Var(Var) {}
PropagationInfo(const CXXBindTemporaryExpr * Tmp)353e5dd7070Spatrick PropagationInfo(const CXXBindTemporaryExpr *Tmp)
354e5dd7070Spatrick : InfoType(IT_Tmp), Tmp(Tmp) {}
355e5dd7070Spatrick
getState() const356e5dd7070Spatrick const ConsumedState &getState() const {
357e5dd7070Spatrick assert(InfoType == IT_State);
358e5dd7070Spatrick return State;
359e5dd7070Spatrick }
360e5dd7070Spatrick
getVarTest() const361e5dd7070Spatrick const VarTestResult &getVarTest() const {
362e5dd7070Spatrick assert(InfoType == IT_VarTest);
363e5dd7070Spatrick return VarTest;
364e5dd7070Spatrick }
365e5dd7070Spatrick
getLTest() const366e5dd7070Spatrick const VarTestResult &getLTest() const {
367e5dd7070Spatrick assert(InfoType == IT_BinTest);
368e5dd7070Spatrick return BinTest.LTest;
369e5dd7070Spatrick }
370e5dd7070Spatrick
getRTest() const371e5dd7070Spatrick const VarTestResult &getRTest() const {
372e5dd7070Spatrick assert(InfoType == IT_BinTest);
373e5dd7070Spatrick return BinTest.RTest;
374e5dd7070Spatrick }
375e5dd7070Spatrick
getVar() const376e5dd7070Spatrick const VarDecl *getVar() const {
377e5dd7070Spatrick assert(InfoType == IT_Var);
378e5dd7070Spatrick return Var;
379e5dd7070Spatrick }
380e5dd7070Spatrick
getTmp() const381e5dd7070Spatrick const CXXBindTemporaryExpr *getTmp() const {
382e5dd7070Spatrick assert(InfoType == IT_Tmp);
383e5dd7070Spatrick return Tmp;
384e5dd7070Spatrick }
385e5dd7070Spatrick
getAsState(const ConsumedStateMap * StateMap) const386e5dd7070Spatrick ConsumedState getAsState(const ConsumedStateMap *StateMap) const {
387e5dd7070Spatrick assert(isVar() || isTmp() || isState());
388e5dd7070Spatrick
389e5dd7070Spatrick if (isVar())
390e5dd7070Spatrick return StateMap->getState(Var);
391e5dd7070Spatrick else if (isTmp())
392e5dd7070Spatrick return StateMap->getState(Tmp);
393e5dd7070Spatrick else if (isState())
394e5dd7070Spatrick return State;
395e5dd7070Spatrick else
396e5dd7070Spatrick return CS_None;
397e5dd7070Spatrick }
398e5dd7070Spatrick
testEffectiveOp() const399e5dd7070Spatrick EffectiveOp testEffectiveOp() const {
400e5dd7070Spatrick assert(InfoType == IT_BinTest);
401e5dd7070Spatrick return BinTest.EOp;
402e5dd7070Spatrick }
403e5dd7070Spatrick
testSourceNode() const404e5dd7070Spatrick const BinaryOperator * testSourceNode() const {
405e5dd7070Spatrick assert(InfoType == IT_BinTest);
406e5dd7070Spatrick return BinTest.Source;
407e5dd7070Spatrick }
408e5dd7070Spatrick
isValid() const409e5dd7070Spatrick bool isValid() const { return InfoType != IT_None; }
isState() const410e5dd7070Spatrick bool isState() const { return InfoType == IT_State; }
isVarTest() const411e5dd7070Spatrick bool isVarTest() const { return InfoType == IT_VarTest; }
isBinTest() const412e5dd7070Spatrick bool isBinTest() const { return InfoType == IT_BinTest; }
isVar() const413e5dd7070Spatrick bool isVar() const { return InfoType == IT_Var; }
isTmp() const414e5dd7070Spatrick bool isTmp() const { return InfoType == IT_Tmp; }
415e5dd7070Spatrick
isTest() const416e5dd7070Spatrick bool isTest() const {
417e5dd7070Spatrick return InfoType == IT_VarTest || InfoType == IT_BinTest;
418e5dd7070Spatrick }
419e5dd7070Spatrick
isPointerToValue() const420e5dd7070Spatrick bool isPointerToValue() const {
421e5dd7070Spatrick return InfoType == IT_Var || InfoType == IT_Tmp;
422e5dd7070Spatrick }
423e5dd7070Spatrick
invertTest() const424e5dd7070Spatrick PropagationInfo invertTest() const {
425e5dd7070Spatrick assert(InfoType == IT_VarTest || InfoType == IT_BinTest);
426e5dd7070Spatrick
427e5dd7070Spatrick if (InfoType == IT_VarTest) {
428e5dd7070Spatrick return PropagationInfo(VarTest.Var,
429e5dd7070Spatrick invertConsumedUnconsumed(VarTest.TestsFor));
430e5dd7070Spatrick
431e5dd7070Spatrick } else if (InfoType == IT_BinTest) {
432e5dd7070Spatrick return PropagationInfo(BinTest.Source,
433e5dd7070Spatrick BinTest.EOp == EO_And ? EO_Or : EO_And,
434e5dd7070Spatrick BinTest.LTest.Var, invertConsumedUnconsumed(BinTest.LTest.TestsFor),
435e5dd7070Spatrick BinTest.RTest.Var, invertConsumedUnconsumed(BinTest.RTest.TestsFor));
436e5dd7070Spatrick } else {
437e5dd7070Spatrick return {};
438e5dd7070Spatrick }
439e5dd7070Spatrick }
440e5dd7070Spatrick };
441e5dd7070Spatrick
442e5dd7070Spatrick } // namespace consumed
443e5dd7070Spatrick } // namespace clang
444e5dd7070Spatrick
445e5dd7070Spatrick static void
setStateForVarOrTmp(ConsumedStateMap * StateMap,const PropagationInfo & PInfo,ConsumedState State)446e5dd7070Spatrick setStateForVarOrTmp(ConsumedStateMap *StateMap, const PropagationInfo &PInfo,
447e5dd7070Spatrick ConsumedState State) {
448e5dd7070Spatrick assert(PInfo.isVar() || PInfo.isTmp());
449e5dd7070Spatrick
450e5dd7070Spatrick if (PInfo.isVar())
451e5dd7070Spatrick StateMap->setState(PInfo.getVar(), State);
452e5dd7070Spatrick else
453e5dd7070Spatrick StateMap->setState(PInfo.getTmp(), State);
454e5dd7070Spatrick }
455e5dd7070Spatrick
456e5dd7070Spatrick namespace clang {
457e5dd7070Spatrick namespace consumed {
458e5dd7070Spatrick
459e5dd7070Spatrick class ConsumedStmtVisitor : public ConstStmtVisitor<ConsumedStmtVisitor> {
460e5dd7070Spatrick using MapType = llvm::DenseMap<const Stmt *, PropagationInfo>;
461e5dd7070Spatrick using PairType= std::pair<const Stmt *, PropagationInfo>;
462e5dd7070Spatrick using InfoEntry = MapType::iterator;
463e5dd7070Spatrick using ConstInfoEntry = MapType::const_iterator;
464e5dd7070Spatrick
465e5dd7070Spatrick ConsumedAnalyzer &Analyzer;
466e5dd7070Spatrick ConsumedStateMap *StateMap;
467e5dd7070Spatrick MapType PropagationMap;
468e5dd7070Spatrick
findInfo(const Expr * E)469e5dd7070Spatrick InfoEntry findInfo(const Expr *E) {
470e5dd7070Spatrick if (const auto Cleanups = dyn_cast<ExprWithCleanups>(E))
471e5dd7070Spatrick if (!Cleanups->cleanupsHaveSideEffects())
472e5dd7070Spatrick E = Cleanups->getSubExpr();
473e5dd7070Spatrick return PropagationMap.find(E->IgnoreParens());
474e5dd7070Spatrick }
475e5dd7070Spatrick
findInfo(const Expr * E) const476e5dd7070Spatrick ConstInfoEntry findInfo(const Expr *E) const {
477e5dd7070Spatrick if (const auto Cleanups = dyn_cast<ExprWithCleanups>(E))
478e5dd7070Spatrick if (!Cleanups->cleanupsHaveSideEffects())
479e5dd7070Spatrick E = Cleanups->getSubExpr();
480e5dd7070Spatrick return PropagationMap.find(E->IgnoreParens());
481e5dd7070Spatrick }
482e5dd7070Spatrick
insertInfo(const Expr * E,const PropagationInfo & PI)483e5dd7070Spatrick void insertInfo(const Expr *E, const PropagationInfo &PI) {
484e5dd7070Spatrick PropagationMap.insert(PairType(E->IgnoreParens(), PI));
485e5dd7070Spatrick }
486e5dd7070Spatrick
487e5dd7070Spatrick void forwardInfo(const Expr *From, const Expr *To);
488e5dd7070Spatrick void copyInfo(const Expr *From, const Expr *To, ConsumedState CS);
489e5dd7070Spatrick ConsumedState getInfo(const Expr *From);
490e5dd7070Spatrick void setInfo(const Expr *To, ConsumedState NS);
491e5dd7070Spatrick void propagateReturnType(const Expr *Call, const FunctionDecl *Fun);
492e5dd7070Spatrick
493e5dd7070Spatrick public:
494e5dd7070Spatrick void checkCallability(const PropagationInfo &PInfo,
495e5dd7070Spatrick const FunctionDecl *FunDecl,
496e5dd7070Spatrick SourceLocation BlameLoc);
497e5dd7070Spatrick bool handleCall(const CallExpr *Call, const Expr *ObjArg,
498e5dd7070Spatrick const FunctionDecl *FunD);
499e5dd7070Spatrick
500e5dd7070Spatrick void VisitBinaryOperator(const BinaryOperator *BinOp);
501e5dd7070Spatrick void VisitCallExpr(const CallExpr *Call);
502e5dd7070Spatrick void VisitCastExpr(const CastExpr *Cast);
503e5dd7070Spatrick void VisitCXXBindTemporaryExpr(const CXXBindTemporaryExpr *Temp);
504e5dd7070Spatrick void VisitCXXConstructExpr(const CXXConstructExpr *Call);
505e5dd7070Spatrick void VisitCXXMemberCallExpr(const CXXMemberCallExpr *Call);
506e5dd7070Spatrick void VisitCXXOperatorCallExpr(const CXXOperatorCallExpr *Call);
507e5dd7070Spatrick void VisitDeclRefExpr(const DeclRefExpr *DeclRef);
508e5dd7070Spatrick void VisitDeclStmt(const DeclStmt *DelcS);
509e5dd7070Spatrick void VisitMaterializeTemporaryExpr(const MaterializeTemporaryExpr *Temp);
510e5dd7070Spatrick void VisitMemberExpr(const MemberExpr *MExpr);
511e5dd7070Spatrick void VisitParmVarDecl(const ParmVarDecl *Param);
512e5dd7070Spatrick void VisitReturnStmt(const ReturnStmt *Ret);
513e5dd7070Spatrick void VisitUnaryOperator(const UnaryOperator *UOp);
514e5dd7070Spatrick void VisitVarDecl(const VarDecl *Var);
515e5dd7070Spatrick
ConsumedStmtVisitor(ConsumedAnalyzer & Analyzer,ConsumedStateMap * StateMap)516e5dd7070Spatrick ConsumedStmtVisitor(ConsumedAnalyzer &Analyzer, ConsumedStateMap *StateMap)
517e5dd7070Spatrick : Analyzer(Analyzer), StateMap(StateMap) {}
518e5dd7070Spatrick
getInfo(const Expr * StmtNode) const519e5dd7070Spatrick PropagationInfo getInfo(const Expr *StmtNode) const {
520e5dd7070Spatrick ConstInfoEntry Entry = findInfo(StmtNode);
521e5dd7070Spatrick
522e5dd7070Spatrick if (Entry != PropagationMap.end())
523e5dd7070Spatrick return Entry->second;
524e5dd7070Spatrick else
525e5dd7070Spatrick return {};
526e5dd7070Spatrick }
527e5dd7070Spatrick
reset(ConsumedStateMap * NewStateMap)528e5dd7070Spatrick void reset(ConsumedStateMap *NewStateMap) {
529e5dd7070Spatrick StateMap = NewStateMap;
530e5dd7070Spatrick }
531e5dd7070Spatrick };
532e5dd7070Spatrick
533e5dd7070Spatrick } // namespace consumed
534e5dd7070Spatrick } // namespace clang
535e5dd7070Spatrick
forwardInfo(const Expr * From,const Expr * To)536e5dd7070Spatrick void ConsumedStmtVisitor::forwardInfo(const Expr *From, const Expr *To) {
537e5dd7070Spatrick InfoEntry Entry = findInfo(From);
538e5dd7070Spatrick if (Entry != PropagationMap.end())
539e5dd7070Spatrick insertInfo(To, Entry->second);
540e5dd7070Spatrick }
541e5dd7070Spatrick
542e5dd7070Spatrick // Create a new state for To, which is initialized to the state of From.
543e5dd7070Spatrick // If NS is not CS_None, sets the state of From to NS.
copyInfo(const Expr * From,const Expr * To,ConsumedState NS)544e5dd7070Spatrick void ConsumedStmtVisitor::copyInfo(const Expr *From, const Expr *To,
545e5dd7070Spatrick ConsumedState NS) {
546e5dd7070Spatrick InfoEntry Entry = findInfo(From);
547e5dd7070Spatrick if (Entry != PropagationMap.end()) {
548e5dd7070Spatrick PropagationInfo& PInfo = Entry->second;
549e5dd7070Spatrick ConsumedState CS = PInfo.getAsState(StateMap);
550e5dd7070Spatrick if (CS != CS_None)
551e5dd7070Spatrick insertInfo(To, PropagationInfo(CS));
552e5dd7070Spatrick if (NS != CS_None && PInfo.isPointerToValue())
553e5dd7070Spatrick setStateForVarOrTmp(StateMap, PInfo, NS);
554e5dd7070Spatrick }
555e5dd7070Spatrick }
556e5dd7070Spatrick
557e5dd7070Spatrick // Get the ConsumedState for From
getInfo(const Expr * From)558e5dd7070Spatrick ConsumedState ConsumedStmtVisitor::getInfo(const Expr *From) {
559e5dd7070Spatrick InfoEntry Entry = findInfo(From);
560e5dd7070Spatrick if (Entry != PropagationMap.end()) {
561e5dd7070Spatrick PropagationInfo& PInfo = Entry->second;
562e5dd7070Spatrick return PInfo.getAsState(StateMap);
563e5dd7070Spatrick }
564e5dd7070Spatrick return CS_None;
565e5dd7070Spatrick }
566e5dd7070Spatrick
567e5dd7070Spatrick // If we already have info for To then update it, otherwise create a new entry.
setInfo(const Expr * To,ConsumedState NS)568e5dd7070Spatrick void ConsumedStmtVisitor::setInfo(const Expr *To, ConsumedState NS) {
569e5dd7070Spatrick InfoEntry Entry = findInfo(To);
570e5dd7070Spatrick if (Entry != PropagationMap.end()) {
571e5dd7070Spatrick PropagationInfo& PInfo = Entry->second;
572e5dd7070Spatrick if (PInfo.isPointerToValue())
573e5dd7070Spatrick setStateForVarOrTmp(StateMap, PInfo, NS);
574e5dd7070Spatrick } else if (NS != CS_None) {
575e5dd7070Spatrick insertInfo(To, PropagationInfo(NS));
576e5dd7070Spatrick }
577e5dd7070Spatrick }
578e5dd7070Spatrick
checkCallability(const PropagationInfo & PInfo,const FunctionDecl * FunDecl,SourceLocation BlameLoc)579e5dd7070Spatrick void ConsumedStmtVisitor::checkCallability(const PropagationInfo &PInfo,
580e5dd7070Spatrick const FunctionDecl *FunDecl,
581e5dd7070Spatrick SourceLocation BlameLoc) {
582e5dd7070Spatrick assert(!PInfo.isTest());
583e5dd7070Spatrick
584e5dd7070Spatrick const CallableWhenAttr *CWAttr = FunDecl->getAttr<CallableWhenAttr>();
585e5dd7070Spatrick if (!CWAttr)
586e5dd7070Spatrick return;
587e5dd7070Spatrick
588e5dd7070Spatrick if (PInfo.isVar()) {
589e5dd7070Spatrick ConsumedState VarState = StateMap->getState(PInfo.getVar());
590e5dd7070Spatrick
591e5dd7070Spatrick if (VarState == CS_None || isCallableInState(CWAttr, VarState))
592e5dd7070Spatrick return;
593e5dd7070Spatrick
594e5dd7070Spatrick Analyzer.WarningsHandler.warnUseInInvalidState(
595e5dd7070Spatrick FunDecl->getNameAsString(), PInfo.getVar()->getNameAsString(),
596e5dd7070Spatrick stateToString(VarState), BlameLoc);
597e5dd7070Spatrick } else {
598e5dd7070Spatrick ConsumedState TmpState = PInfo.getAsState(StateMap);
599e5dd7070Spatrick
600e5dd7070Spatrick if (TmpState == CS_None || isCallableInState(CWAttr, TmpState))
601e5dd7070Spatrick return;
602e5dd7070Spatrick
603e5dd7070Spatrick Analyzer.WarningsHandler.warnUseOfTempInInvalidState(
604e5dd7070Spatrick FunDecl->getNameAsString(), stateToString(TmpState), BlameLoc);
605e5dd7070Spatrick }
606e5dd7070Spatrick }
607e5dd7070Spatrick
608e5dd7070Spatrick // Factors out common behavior for function, method, and operator calls.
609e5dd7070Spatrick // Check parameters and set parameter state if necessary.
610e5dd7070Spatrick // Returns true if the state of ObjArg is set, or false otherwise.
handleCall(const CallExpr * Call,const Expr * ObjArg,const FunctionDecl * FunD)611e5dd7070Spatrick bool ConsumedStmtVisitor::handleCall(const CallExpr *Call, const Expr *ObjArg,
612e5dd7070Spatrick const FunctionDecl *FunD) {
613e5dd7070Spatrick unsigned Offset = 0;
614e5dd7070Spatrick if (isa<CXXOperatorCallExpr>(Call) && isa<CXXMethodDecl>(FunD))
615e5dd7070Spatrick Offset = 1; // first argument is 'this'
616e5dd7070Spatrick
617e5dd7070Spatrick // check explicit parameters
618e5dd7070Spatrick for (unsigned Index = Offset; Index < Call->getNumArgs(); ++Index) {
619e5dd7070Spatrick // Skip variable argument lists.
620e5dd7070Spatrick if (Index - Offset >= FunD->getNumParams())
621e5dd7070Spatrick break;
622e5dd7070Spatrick
623e5dd7070Spatrick const ParmVarDecl *Param = FunD->getParamDecl(Index - Offset);
624e5dd7070Spatrick QualType ParamType = Param->getType();
625e5dd7070Spatrick
626e5dd7070Spatrick InfoEntry Entry = findInfo(Call->getArg(Index));
627e5dd7070Spatrick
628e5dd7070Spatrick if (Entry == PropagationMap.end() || Entry->second.isTest())
629e5dd7070Spatrick continue;
630e5dd7070Spatrick PropagationInfo PInfo = Entry->second;
631e5dd7070Spatrick
632e5dd7070Spatrick // Check that the parameter is in the correct state.
633e5dd7070Spatrick if (ParamTypestateAttr *PTA = Param->getAttr<ParamTypestateAttr>()) {
634e5dd7070Spatrick ConsumedState ParamState = PInfo.getAsState(StateMap);
635e5dd7070Spatrick ConsumedState ExpectedState = mapParamTypestateAttrState(PTA);
636e5dd7070Spatrick
637e5dd7070Spatrick if (ParamState != ExpectedState)
638e5dd7070Spatrick Analyzer.WarningsHandler.warnParamTypestateMismatch(
639e5dd7070Spatrick Call->getArg(Index)->getExprLoc(),
640e5dd7070Spatrick stateToString(ExpectedState), stateToString(ParamState));
641e5dd7070Spatrick }
642e5dd7070Spatrick
643e5dd7070Spatrick if (!(Entry->second.isVar() || Entry->second.isTmp()))
644e5dd7070Spatrick continue;
645e5dd7070Spatrick
646e5dd7070Spatrick // Adjust state on the caller side.
647e5dd7070Spatrick if (ReturnTypestateAttr *RT = Param->getAttr<ReturnTypestateAttr>())
648e5dd7070Spatrick setStateForVarOrTmp(StateMap, PInfo, mapReturnTypestateAttrState(RT));
649e5dd7070Spatrick else if (isRValueRef(ParamType) || isConsumableType(ParamType))
650e5dd7070Spatrick setStateForVarOrTmp(StateMap, PInfo, consumed::CS_Consumed);
651e5dd7070Spatrick else if (isPointerOrRef(ParamType) &&
652e5dd7070Spatrick (!ParamType->getPointeeType().isConstQualified() ||
653e5dd7070Spatrick isSetOnReadPtrType(ParamType)))
654e5dd7070Spatrick setStateForVarOrTmp(StateMap, PInfo, consumed::CS_Unknown);
655e5dd7070Spatrick }
656e5dd7070Spatrick
657e5dd7070Spatrick if (!ObjArg)
658e5dd7070Spatrick return false;
659e5dd7070Spatrick
660e5dd7070Spatrick // check implicit 'self' parameter, if present
661e5dd7070Spatrick InfoEntry Entry = findInfo(ObjArg);
662e5dd7070Spatrick if (Entry != PropagationMap.end()) {
663e5dd7070Spatrick PropagationInfo PInfo = Entry->second;
664e5dd7070Spatrick checkCallability(PInfo, FunD, Call->getExprLoc());
665e5dd7070Spatrick
666e5dd7070Spatrick if (SetTypestateAttr *STA = FunD->getAttr<SetTypestateAttr>()) {
667e5dd7070Spatrick if (PInfo.isVar()) {
668e5dd7070Spatrick StateMap->setState(PInfo.getVar(), mapSetTypestateAttrState(STA));
669e5dd7070Spatrick return true;
670e5dd7070Spatrick }
671e5dd7070Spatrick else if (PInfo.isTmp()) {
672e5dd7070Spatrick StateMap->setState(PInfo.getTmp(), mapSetTypestateAttrState(STA));
673e5dd7070Spatrick return true;
674e5dd7070Spatrick }
675e5dd7070Spatrick }
676e5dd7070Spatrick else if (isTestingFunction(FunD) && PInfo.isVar()) {
677e5dd7070Spatrick PropagationMap.insert(PairType(Call,
678e5dd7070Spatrick PropagationInfo(PInfo.getVar(), testsFor(FunD))));
679e5dd7070Spatrick }
680e5dd7070Spatrick }
681e5dd7070Spatrick return false;
682e5dd7070Spatrick }
683e5dd7070Spatrick
propagateReturnType(const Expr * Call,const FunctionDecl * Fun)684e5dd7070Spatrick void ConsumedStmtVisitor::propagateReturnType(const Expr *Call,
685e5dd7070Spatrick const FunctionDecl *Fun) {
686e5dd7070Spatrick QualType RetType = Fun->getCallResultType();
687e5dd7070Spatrick if (RetType->isReferenceType())
688e5dd7070Spatrick RetType = RetType->getPointeeType();
689e5dd7070Spatrick
690e5dd7070Spatrick if (isConsumableType(RetType)) {
691e5dd7070Spatrick ConsumedState ReturnState;
692e5dd7070Spatrick if (ReturnTypestateAttr *RTA = Fun->getAttr<ReturnTypestateAttr>())
693e5dd7070Spatrick ReturnState = mapReturnTypestateAttrState(RTA);
694e5dd7070Spatrick else
695e5dd7070Spatrick ReturnState = mapConsumableAttrState(RetType);
696e5dd7070Spatrick
697e5dd7070Spatrick PropagationMap.insert(PairType(Call, PropagationInfo(ReturnState)));
698e5dd7070Spatrick }
699e5dd7070Spatrick }
700e5dd7070Spatrick
VisitBinaryOperator(const BinaryOperator * BinOp)701e5dd7070Spatrick void ConsumedStmtVisitor::VisitBinaryOperator(const BinaryOperator *BinOp) {
702e5dd7070Spatrick switch (BinOp->getOpcode()) {
703e5dd7070Spatrick case BO_LAnd:
704e5dd7070Spatrick case BO_LOr : {
705e5dd7070Spatrick InfoEntry LEntry = findInfo(BinOp->getLHS()),
706e5dd7070Spatrick REntry = findInfo(BinOp->getRHS());
707e5dd7070Spatrick
708e5dd7070Spatrick VarTestResult LTest, RTest;
709e5dd7070Spatrick
710e5dd7070Spatrick if (LEntry != PropagationMap.end() && LEntry->second.isVarTest()) {
711e5dd7070Spatrick LTest = LEntry->second.getVarTest();
712e5dd7070Spatrick } else {
713e5dd7070Spatrick LTest.Var = nullptr;
714e5dd7070Spatrick LTest.TestsFor = CS_None;
715e5dd7070Spatrick }
716e5dd7070Spatrick
717e5dd7070Spatrick if (REntry != PropagationMap.end() && REntry->second.isVarTest()) {
718e5dd7070Spatrick RTest = REntry->second.getVarTest();
719e5dd7070Spatrick } else {
720e5dd7070Spatrick RTest.Var = nullptr;
721e5dd7070Spatrick RTest.TestsFor = CS_None;
722e5dd7070Spatrick }
723e5dd7070Spatrick
724e5dd7070Spatrick if (!(LTest.Var == nullptr && RTest.Var == nullptr))
725e5dd7070Spatrick PropagationMap.insert(PairType(BinOp, PropagationInfo(BinOp,
726e5dd7070Spatrick static_cast<EffectiveOp>(BinOp->getOpcode() == BO_LOr), LTest, RTest)));
727e5dd7070Spatrick break;
728e5dd7070Spatrick }
729e5dd7070Spatrick
730e5dd7070Spatrick case BO_PtrMemD:
731e5dd7070Spatrick case BO_PtrMemI:
732e5dd7070Spatrick forwardInfo(BinOp->getLHS(), BinOp);
733e5dd7070Spatrick break;
734e5dd7070Spatrick
735e5dd7070Spatrick default:
736e5dd7070Spatrick break;
737e5dd7070Spatrick }
738e5dd7070Spatrick }
739e5dd7070Spatrick
VisitCallExpr(const CallExpr * Call)740e5dd7070Spatrick void ConsumedStmtVisitor::VisitCallExpr(const CallExpr *Call) {
741e5dd7070Spatrick const FunctionDecl *FunDecl = Call->getDirectCallee();
742e5dd7070Spatrick if (!FunDecl)
743e5dd7070Spatrick return;
744e5dd7070Spatrick
745e5dd7070Spatrick // Special case for the std::move function.
746e5dd7070Spatrick // TODO: Make this more specific. (Deferred)
747e5dd7070Spatrick if (Call->isCallToStdMove()) {
748e5dd7070Spatrick copyInfo(Call->getArg(0), Call, CS_Consumed);
749e5dd7070Spatrick return;
750e5dd7070Spatrick }
751e5dd7070Spatrick
752e5dd7070Spatrick handleCall(Call, nullptr, FunDecl);
753e5dd7070Spatrick propagateReturnType(Call, FunDecl);
754e5dd7070Spatrick }
755e5dd7070Spatrick
VisitCastExpr(const CastExpr * Cast)756e5dd7070Spatrick void ConsumedStmtVisitor::VisitCastExpr(const CastExpr *Cast) {
757e5dd7070Spatrick forwardInfo(Cast->getSubExpr(), Cast);
758e5dd7070Spatrick }
759e5dd7070Spatrick
VisitCXXBindTemporaryExpr(const CXXBindTemporaryExpr * Temp)760e5dd7070Spatrick void ConsumedStmtVisitor::VisitCXXBindTemporaryExpr(
761e5dd7070Spatrick const CXXBindTemporaryExpr *Temp) {
762e5dd7070Spatrick
763e5dd7070Spatrick InfoEntry Entry = findInfo(Temp->getSubExpr());
764e5dd7070Spatrick
765e5dd7070Spatrick if (Entry != PropagationMap.end() && !Entry->second.isTest()) {
766e5dd7070Spatrick StateMap->setState(Temp, Entry->second.getAsState(StateMap));
767e5dd7070Spatrick PropagationMap.insert(PairType(Temp, PropagationInfo(Temp)));
768e5dd7070Spatrick }
769e5dd7070Spatrick }
770e5dd7070Spatrick
VisitCXXConstructExpr(const CXXConstructExpr * Call)771e5dd7070Spatrick void ConsumedStmtVisitor::VisitCXXConstructExpr(const CXXConstructExpr *Call) {
772e5dd7070Spatrick CXXConstructorDecl *Constructor = Call->getConstructor();
773e5dd7070Spatrick
774e5dd7070Spatrick QualType ThisType = Constructor->getThisType()->getPointeeType();
775e5dd7070Spatrick
776e5dd7070Spatrick if (!isConsumableType(ThisType))
777e5dd7070Spatrick return;
778e5dd7070Spatrick
779e5dd7070Spatrick // FIXME: What should happen if someone annotates the move constructor?
780e5dd7070Spatrick if (ReturnTypestateAttr *RTA = Constructor->getAttr<ReturnTypestateAttr>()) {
781e5dd7070Spatrick // TODO: Adjust state of args appropriately.
782e5dd7070Spatrick ConsumedState RetState = mapReturnTypestateAttrState(RTA);
783e5dd7070Spatrick PropagationMap.insert(PairType(Call, PropagationInfo(RetState)));
784e5dd7070Spatrick } else if (Constructor->isDefaultConstructor()) {
785e5dd7070Spatrick PropagationMap.insert(PairType(Call,
786e5dd7070Spatrick PropagationInfo(consumed::CS_Consumed)));
787e5dd7070Spatrick } else if (Constructor->isMoveConstructor()) {
788e5dd7070Spatrick copyInfo(Call->getArg(0), Call, CS_Consumed);
789e5dd7070Spatrick } else if (Constructor->isCopyConstructor()) {
790e5dd7070Spatrick // Copy state from arg. If setStateOnRead then set arg to CS_Unknown.
791e5dd7070Spatrick ConsumedState NS =
792e5dd7070Spatrick isSetOnReadPtrType(Constructor->getThisType()) ?
793e5dd7070Spatrick CS_Unknown : CS_None;
794e5dd7070Spatrick copyInfo(Call->getArg(0), Call, NS);
795e5dd7070Spatrick } else {
796e5dd7070Spatrick // TODO: Adjust state of args appropriately.
797e5dd7070Spatrick ConsumedState RetState = mapConsumableAttrState(ThisType);
798e5dd7070Spatrick PropagationMap.insert(PairType(Call, PropagationInfo(RetState)));
799e5dd7070Spatrick }
800e5dd7070Spatrick }
801e5dd7070Spatrick
VisitCXXMemberCallExpr(const CXXMemberCallExpr * Call)802e5dd7070Spatrick void ConsumedStmtVisitor::VisitCXXMemberCallExpr(
803e5dd7070Spatrick const CXXMemberCallExpr *Call) {
804e5dd7070Spatrick CXXMethodDecl* MD = Call->getMethodDecl();
805e5dd7070Spatrick if (!MD)
806e5dd7070Spatrick return;
807e5dd7070Spatrick
808e5dd7070Spatrick handleCall(Call, Call->getImplicitObjectArgument(), MD);
809e5dd7070Spatrick propagateReturnType(Call, MD);
810e5dd7070Spatrick }
811e5dd7070Spatrick
VisitCXXOperatorCallExpr(const CXXOperatorCallExpr * Call)812e5dd7070Spatrick void ConsumedStmtVisitor::VisitCXXOperatorCallExpr(
813e5dd7070Spatrick const CXXOperatorCallExpr *Call) {
814e5dd7070Spatrick const auto *FunDecl = dyn_cast_or_null<FunctionDecl>(Call->getDirectCallee());
815e5dd7070Spatrick if (!FunDecl) return;
816e5dd7070Spatrick
817e5dd7070Spatrick if (Call->getOperator() == OO_Equal) {
818e5dd7070Spatrick ConsumedState CS = getInfo(Call->getArg(1));
819e5dd7070Spatrick if (!handleCall(Call, Call->getArg(0), FunDecl))
820e5dd7070Spatrick setInfo(Call->getArg(0), CS);
821e5dd7070Spatrick return;
822e5dd7070Spatrick }
823e5dd7070Spatrick
824e5dd7070Spatrick if (const auto *MCall = dyn_cast<CXXMemberCallExpr>(Call))
825e5dd7070Spatrick handleCall(MCall, MCall->getImplicitObjectArgument(), FunDecl);
826e5dd7070Spatrick else
827e5dd7070Spatrick handleCall(Call, Call->getArg(0), FunDecl);
828e5dd7070Spatrick
829e5dd7070Spatrick propagateReturnType(Call, FunDecl);
830e5dd7070Spatrick }
831e5dd7070Spatrick
VisitDeclRefExpr(const DeclRefExpr * DeclRef)832e5dd7070Spatrick void ConsumedStmtVisitor::VisitDeclRefExpr(const DeclRefExpr *DeclRef) {
833e5dd7070Spatrick if (const auto *Var = dyn_cast_or_null<VarDecl>(DeclRef->getDecl()))
834e5dd7070Spatrick if (StateMap->getState(Var) != consumed::CS_None)
835e5dd7070Spatrick PropagationMap.insert(PairType(DeclRef, PropagationInfo(Var)));
836e5dd7070Spatrick }
837e5dd7070Spatrick
VisitDeclStmt(const DeclStmt * DeclS)838e5dd7070Spatrick void ConsumedStmtVisitor::VisitDeclStmt(const DeclStmt *DeclS) {
839e5dd7070Spatrick for (const auto *DI : DeclS->decls())
840e5dd7070Spatrick if (isa<VarDecl>(DI))
841e5dd7070Spatrick VisitVarDecl(cast<VarDecl>(DI));
842e5dd7070Spatrick
843e5dd7070Spatrick if (DeclS->isSingleDecl())
844e5dd7070Spatrick if (const auto *Var = dyn_cast_or_null<VarDecl>(DeclS->getSingleDecl()))
845e5dd7070Spatrick PropagationMap.insert(PairType(DeclS, PropagationInfo(Var)));
846e5dd7070Spatrick }
847e5dd7070Spatrick
VisitMaterializeTemporaryExpr(const MaterializeTemporaryExpr * Temp)848e5dd7070Spatrick void ConsumedStmtVisitor::VisitMaterializeTemporaryExpr(
849e5dd7070Spatrick const MaterializeTemporaryExpr *Temp) {
850e5dd7070Spatrick forwardInfo(Temp->getSubExpr(), Temp);
851e5dd7070Spatrick }
852e5dd7070Spatrick
VisitMemberExpr(const MemberExpr * MExpr)853e5dd7070Spatrick void ConsumedStmtVisitor::VisitMemberExpr(const MemberExpr *MExpr) {
854e5dd7070Spatrick forwardInfo(MExpr->getBase(), MExpr);
855e5dd7070Spatrick }
856e5dd7070Spatrick
VisitParmVarDecl(const ParmVarDecl * Param)857e5dd7070Spatrick void ConsumedStmtVisitor::VisitParmVarDecl(const ParmVarDecl *Param) {
858e5dd7070Spatrick QualType ParamType = Param->getType();
859e5dd7070Spatrick ConsumedState ParamState = consumed::CS_None;
860e5dd7070Spatrick
861e5dd7070Spatrick if (const ParamTypestateAttr *PTA = Param->getAttr<ParamTypestateAttr>())
862e5dd7070Spatrick ParamState = mapParamTypestateAttrState(PTA);
863e5dd7070Spatrick else if (isConsumableType(ParamType))
864e5dd7070Spatrick ParamState = mapConsumableAttrState(ParamType);
865e5dd7070Spatrick else if (isRValueRef(ParamType) &&
866e5dd7070Spatrick isConsumableType(ParamType->getPointeeType()))
867e5dd7070Spatrick ParamState = mapConsumableAttrState(ParamType->getPointeeType());
868e5dd7070Spatrick else if (ParamType->isReferenceType() &&
869e5dd7070Spatrick isConsumableType(ParamType->getPointeeType()))
870e5dd7070Spatrick ParamState = consumed::CS_Unknown;
871e5dd7070Spatrick
872e5dd7070Spatrick if (ParamState != CS_None)
873e5dd7070Spatrick StateMap->setState(Param, ParamState);
874e5dd7070Spatrick }
875e5dd7070Spatrick
VisitReturnStmt(const ReturnStmt * Ret)876e5dd7070Spatrick void ConsumedStmtVisitor::VisitReturnStmt(const ReturnStmt *Ret) {
877e5dd7070Spatrick ConsumedState ExpectedState = Analyzer.getExpectedReturnState();
878e5dd7070Spatrick
879e5dd7070Spatrick if (ExpectedState != CS_None) {
880e5dd7070Spatrick InfoEntry Entry = findInfo(Ret->getRetValue());
881e5dd7070Spatrick
882e5dd7070Spatrick if (Entry != PropagationMap.end()) {
883e5dd7070Spatrick ConsumedState RetState = Entry->second.getAsState(StateMap);
884e5dd7070Spatrick
885e5dd7070Spatrick if (RetState != ExpectedState)
886e5dd7070Spatrick Analyzer.WarningsHandler.warnReturnTypestateMismatch(
887e5dd7070Spatrick Ret->getReturnLoc(), stateToString(ExpectedState),
888e5dd7070Spatrick stateToString(RetState));
889e5dd7070Spatrick }
890e5dd7070Spatrick }
891e5dd7070Spatrick
892e5dd7070Spatrick StateMap->checkParamsForReturnTypestate(Ret->getBeginLoc(),
893e5dd7070Spatrick Analyzer.WarningsHandler);
894e5dd7070Spatrick }
895e5dd7070Spatrick
VisitUnaryOperator(const UnaryOperator * UOp)896e5dd7070Spatrick void ConsumedStmtVisitor::VisitUnaryOperator(const UnaryOperator *UOp) {
897e5dd7070Spatrick InfoEntry Entry = findInfo(UOp->getSubExpr());
898e5dd7070Spatrick if (Entry == PropagationMap.end()) return;
899e5dd7070Spatrick
900e5dd7070Spatrick switch (UOp->getOpcode()) {
901e5dd7070Spatrick case UO_AddrOf:
902e5dd7070Spatrick PropagationMap.insert(PairType(UOp, Entry->second));
903e5dd7070Spatrick break;
904e5dd7070Spatrick
905e5dd7070Spatrick case UO_LNot:
906e5dd7070Spatrick if (Entry->second.isTest())
907e5dd7070Spatrick PropagationMap.insert(PairType(UOp, Entry->second.invertTest()));
908e5dd7070Spatrick break;
909e5dd7070Spatrick
910e5dd7070Spatrick default:
911e5dd7070Spatrick break;
912e5dd7070Spatrick }
913e5dd7070Spatrick }
914e5dd7070Spatrick
915e5dd7070Spatrick // TODO: See if I need to check for reference types here.
VisitVarDecl(const VarDecl * Var)916e5dd7070Spatrick void ConsumedStmtVisitor::VisitVarDecl(const VarDecl *Var) {
917e5dd7070Spatrick if (isConsumableType(Var->getType())) {
918e5dd7070Spatrick if (Var->hasInit()) {
919e5dd7070Spatrick MapType::iterator VIT = findInfo(Var->getInit()->IgnoreImplicit());
920e5dd7070Spatrick if (VIT != PropagationMap.end()) {
921e5dd7070Spatrick PropagationInfo PInfo = VIT->second;
922e5dd7070Spatrick ConsumedState St = PInfo.getAsState(StateMap);
923e5dd7070Spatrick
924e5dd7070Spatrick if (St != consumed::CS_None) {
925e5dd7070Spatrick StateMap->setState(Var, St);
926e5dd7070Spatrick return;
927e5dd7070Spatrick }
928e5dd7070Spatrick }
929e5dd7070Spatrick }
930e5dd7070Spatrick // Otherwise
931e5dd7070Spatrick StateMap->setState(Var, consumed::CS_Unknown);
932e5dd7070Spatrick }
933e5dd7070Spatrick }
934e5dd7070Spatrick
splitVarStateForIf(const IfStmt * IfNode,const VarTestResult & Test,ConsumedStateMap * ThenStates,ConsumedStateMap * ElseStates)935e5dd7070Spatrick static void splitVarStateForIf(const IfStmt *IfNode, const VarTestResult &Test,
936e5dd7070Spatrick ConsumedStateMap *ThenStates,
937e5dd7070Spatrick ConsumedStateMap *ElseStates) {
938e5dd7070Spatrick ConsumedState VarState = ThenStates->getState(Test.Var);
939e5dd7070Spatrick
940e5dd7070Spatrick if (VarState == CS_Unknown) {
941e5dd7070Spatrick ThenStates->setState(Test.Var, Test.TestsFor);
942e5dd7070Spatrick ElseStates->setState(Test.Var, invertConsumedUnconsumed(Test.TestsFor));
943e5dd7070Spatrick } else if (VarState == invertConsumedUnconsumed(Test.TestsFor)) {
944e5dd7070Spatrick ThenStates->markUnreachable();
945e5dd7070Spatrick } else if (VarState == Test.TestsFor) {
946e5dd7070Spatrick ElseStates->markUnreachable();
947e5dd7070Spatrick }
948e5dd7070Spatrick }
949e5dd7070Spatrick
splitVarStateForIfBinOp(const PropagationInfo & PInfo,ConsumedStateMap * ThenStates,ConsumedStateMap * ElseStates)950e5dd7070Spatrick static void splitVarStateForIfBinOp(const PropagationInfo &PInfo,
951e5dd7070Spatrick ConsumedStateMap *ThenStates,
952e5dd7070Spatrick ConsumedStateMap *ElseStates) {
953e5dd7070Spatrick const VarTestResult <est = PInfo.getLTest(),
954e5dd7070Spatrick &RTest = PInfo.getRTest();
955e5dd7070Spatrick
956e5dd7070Spatrick ConsumedState LState = LTest.Var ? ThenStates->getState(LTest.Var) : CS_None,
957e5dd7070Spatrick RState = RTest.Var ? ThenStates->getState(RTest.Var) : CS_None;
958e5dd7070Spatrick
959e5dd7070Spatrick if (LTest.Var) {
960e5dd7070Spatrick if (PInfo.testEffectiveOp() == EO_And) {
961e5dd7070Spatrick if (LState == CS_Unknown) {
962e5dd7070Spatrick ThenStates->setState(LTest.Var, LTest.TestsFor);
963e5dd7070Spatrick } else if (LState == invertConsumedUnconsumed(LTest.TestsFor)) {
964e5dd7070Spatrick ThenStates->markUnreachable();
965e5dd7070Spatrick } else if (LState == LTest.TestsFor && isKnownState(RState)) {
966e5dd7070Spatrick if (RState == RTest.TestsFor)
967e5dd7070Spatrick ElseStates->markUnreachable();
968e5dd7070Spatrick else
969e5dd7070Spatrick ThenStates->markUnreachable();
970e5dd7070Spatrick }
971e5dd7070Spatrick } else {
972e5dd7070Spatrick if (LState == CS_Unknown) {
973e5dd7070Spatrick ElseStates->setState(LTest.Var,
974e5dd7070Spatrick invertConsumedUnconsumed(LTest.TestsFor));
975e5dd7070Spatrick } else if (LState == LTest.TestsFor) {
976e5dd7070Spatrick ElseStates->markUnreachable();
977e5dd7070Spatrick } else if (LState == invertConsumedUnconsumed(LTest.TestsFor) &&
978e5dd7070Spatrick isKnownState(RState)) {
979e5dd7070Spatrick if (RState == RTest.TestsFor)
980e5dd7070Spatrick ElseStates->markUnreachable();
981e5dd7070Spatrick else
982e5dd7070Spatrick ThenStates->markUnreachable();
983e5dd7070Spatrick }
984e5dd7070Spatrick }
985e5dd7070Spatrick }
986e5dd7070Spatrick
987e5dd7070Spatrick if (RTest.Var) {
988e5dd7070Spatrick if (PInfo.testEffectiveOp() == EO_And) {
989e5dd7070Spatrick if (RState == CS_Unknown)
990e5dd7070Spatrick ThenStates->setState(RTest.Var, RTest.TestsFor);
991e5dd7070Spatrick else if (RState == invertConsumedUnconsumed(RTest.TestsFor))
992e5dd7070Spatrick ThenStates->markUnreachable();
993e5dd7070Spatrick } else {
994e5dd7070Spatrick if (RState == CS_Unknown)
995e5dd7070Spatrick ElseStates->setState(RTest.Var,
996e5dd7070Spatrick invertConsumedUnconsumed(RTest.TestsFor));
997e5dd7070Spatrick else if (RState == RTest.TestsFor)
998e5dd7070Spatrick ElseStates->markUnreachable();
999e5dd7070Spatrick }
1000e5dd7070Spatrick }
1001e5dd7070Spatrick }
1002e5dd7070Spatrick
allBackEdgesVisited(const CFGBlock * CurrBlock,const CFGBlock * TargetBlock)1003e5dd7070Spatrick bool ConsumedBlockInfo::allBackEdgesVisited(const CFGBlock *CurrBlock,
1004e5dd7070Spatrick const CFGBlock *TargetBlock) {
1005e5dd7070Spatrick assert(CurrBlock && "Block pointer must not be NULL");
1006e5dd7070Spatrick assert(TargetBlock && "TargetBlock pointer must not be NULL");
1007e5dd7070Spatrick
1008e5dd7070Spatrick unsigned int CurrBlockOrder = VisitOrder[CurrBlock->getBlockID()];
1009e5dd7070Spatrick for (CFGBlock::const_pred_iterator PI = TargetBlock->pred_begin(),
1010e5dd7070Spatrick PE = TargetBlock->pred_end(); PI != PE; ++PI) {
1011e5dd7070Spatrick if (*PI && CurrBlockOrder < VisitOrder[(*PI)->getBlockID()] )
1012e5dd7070Spatrick return false;
1013e5dd7070Spatrick }
1014e5dd7070Spatrick return true;
1015e5dd7070Spatrick }
1016e5dd7070Spatrick
addInfo(const CFGBlock * Block,ConsumedStateMap * StateMap,std::unique_ptr<ConsumedStateMap> & OwnedStateMap)1017e5dd7070Spatrick void ConsumedBlockInfo::addInfo(
1018e5dd7070Spatrick const CFGBlock *Block, ConsumedStateMap *StateMap,
1019e5dd7070Spatrick std::unique_ptr<ConsumedStateMap> &OwnedStateMap) {
1020e5dd7070Spatrick assert(Block && "Block pointer must not be NULL");
1021e5dd7070Spatrick
1022e5dd7070Spatrick auto &Entry = StateMapsArray[Block->getBlockID()];
1023e5dd7070Spatrick
1024e5dd7070Spatrick if (Entry) {
1025e5dd7070Spatrick Entry->intersect(*StateMap);
1026e5dd7070Spatrick } else if (OwnedStateMap)
1027e5dd7070Spatrick Entry = std::move(OwnedStateMap);
1028e5dd7070Spatrick else
1029e5dd7070Spatrick Entry = std::make_unique<ConsumedStateMap>(*StateMap);
1030e5dd7070Spatrick }
1031e5dd7070Spatrick
addInfo(const CFGBlock * Block,std::unique_ptr<ConsumedStateMap> StateMap)1032e5dd7070Spatrick void ConsumedBlockInfo::addInfo(const CFGBlock *Block,
1033e5dd7070Spatrick std::unique_ptr<ConsumedStateMap> StateMap) {
1034e5dd7070Spatrick assert(Block && "Block pointer must not be NULL");
1035e5dd7070Spatrick
1036e5dd7070Spatrick auto &Entry = StateMapsArray[Block->getBlockID()];
1037e5dd7070Spatrick
1038e5dd7070Spatrick if (Entry) {
1039e5dd7070Spatrick Entry->intersect(*StateMap);
1040e5dd7070Spatrick } else {
1041e5dd7070Spatrick Entry = std::move(StateMap);
1042e5dd7070Spatrick }
1043e5dd7070Spatrick }
1044e5dd7070Spatrick
borrowInfo(const CFGBlock * Block)1045e5dd7070Spatrick ConsumedStateMap* ConsumedBlockInfo::borrowInfo(const CFGBlock *Block) {
1046e5dd7070Spatrick assert(Block && "Block pointer must not be NULL");
1047e5dd7070Spatrick assert(StateMapsArray[Block->getBlockID()] && "Block has no block info");
1048e5dd7070Spatrick
1049e5dd7070Spatrick return StateMapsArray[Block->getBlockID()].get();
1050e5dd7070Spatrick }
1051e5dd7070Spatrick
discardInfo(const CFGBlock * Block)1052e5dd7070Spatrick void ConsumedBlockInfo::discardInfo(const CFGBlock *Block) {
1053e5dd7070Spatrick StateMapsArray[Block->getBlockID()] = nullptr;
1054e5dd7070Spatrick }
1055e5dd7070Spatrick
1056e5dd7070Spatrick std::unique_ptr<ConsumedStateMap>
getInfo(const CFGBlock * Block)1057e5dd7070Spatrick ConsumedBlockInfo::getInfo(const CFGBlock *Block) {
1058e5dd7070Spatrick assert(Block && "Block pointer must not be NULL");
1059e5dd7070Spatrick
1060e5dd7070Spatrick auto &Entry = StateMapsArray[Block->getBlockID()];
1061e5dd7070Spatrick return isBackEdgeTarget(Block) ? std::make_unique<ConsumedStateMap>(*Entry)
1062e5dd7070Spatrick : std::move(Entry);
1063e5dd7070Spatrick }
1064e5dd7070Spatrick
isBackEdge(const CFGBlock * From,const CFGBlock * To)1065e5dd7070Spatrick bool ConsumedBlockInfo::isBackEdge(const CFGBlock *From, const CFGBlock *To) {
1066e5dd7070Spatrick assert(From && "From block must not be NULL");
1067e5dd7070Spatrick assert(To && "From block must not be NULL");
1068e5dd7070Spatrick
1069e5dd7070Spatrick return VisitOrder[From->getBlockID()] > VisitOrder[To->getBlockID()];
1070e5dd7070Spatrick }
1071e5dd7070Spatrick
isBackEdgeTarget(const CFGBlock * Block)1072e5dd7070Spatrick bool ConsumedBlockInfo::isBackEdgeTarget(const CFGBlock *Block) {
1073e5dd7070Spatrick assert(Block && "Block pointer must not be NULL");
1074e5dd7070Spatrick
1075e5dd7070Spatrick // Anything with less than two predecessors can't be the target of a back
1076e5dd7070Spatrick // edge.
1077e5dd7070Spatrick if (Block->pred_size() < 2)
1078e5dd7070Spatrick return false;
1079e5dd7070Spatrick
1080e5dd7070Spatrick unsigned int BlockVisitOrder = VisitOrder[Block->getBlockID()];
1081e5dd7070Spatrick for (CFGBlock::const_pred_iterator PI = Block->pred_begin(),
1082e5dd7070Spatrick PE = Block->pred_end(); PI != PE; ++PI) {
1083e5dd7070Spatrick if (*PI && BlockVisitOrder < VisitOrder[(*PI)->getBlockID()])
1084e5dd7070Spatrick return true;
1085e5dd7070Spatrick }
1086e5dd7070Spatrick return false;
1087e5dd7070Spatrick }
1088e5dd7070Spatrick
checkParamsForReturnTypestate(SourceLocation BlameLoc,ConsumedWarningsHandlerBase & WarningsHandler) const1089e5dd7070Spatrick void ConsumedStateMap::checkParamsForReturnTypestate(SourceLocation BlameLoc,
1090e5dd7070Spatrick ConsumedWarningsHandlerBase &WarningsHandler) const {
1091e5dd7070Spatrick
1092e5dd7070Spatrick for (const auto &DM : VarMap) {
1093e5dd7070Spatrick if (isa<ParmVarDecl>(DM.first)) {
1094e5dd7070Spatrick const auto *Param = cast<ParmVarDecl>(DM.first);
1095e5dd7070Spatrick const ReturnTypestateAttr *RTA = Param->getAttr<ReturnTypestateAttr>();
1096e5dd7070Spatrick
1097e5dd7070Spatrick if (!RTA)
1098e5dd7070Spatrick continue;
1099e5dd7070Spatrick
1100e5dd7070Spatrick ConsumedState ExpectedState = mapReturnTypestateAttrState(RTA);
1101e5dd7070Spatrick if (DM.second != ExpectedState)
1102e5dd7070Spatrick WarningsHandler.warnParamReturnTypestateMismatch(BlameLoc,
1103e5dd7070Spatrick Param->getNameAsString(), stateToString(ExpectedState),
1104e5dd7070Spatrick stateToString(DM.second));
1105e5dd7070Spatrick }
1106e5dd7070Spatrick }
1107e5dd7070Spatrick }
1108e5dd7070Spatrick
clearTemporaries()1109e5dd7070Spatrick void ConsumedStateMap::clearTemporaries() {
1110e5dd7070Spatrick TmpMap.clear();
1111e5dd7070Spatrick }
1112e5dd7070Spatrick
getState(const VarDecl * Var) const1113e5dd7070Spatrick ConsumedState ConsumedStateMap::getState(const VarDecl *Var) const {
1114e5dd7070Spatrick VarMapType::const_iterator Entry = VarMap.find(Var);
1115e5dd7070Spatrick
1116e5dd7070Spatrick if (Entry != VarMap.end())
1117e5dd7070Spatrick return Entry->second;
1118e5dd7070Spatrick
1119e5dd7070Spatrick return CS_None;
1120e5dd7070Spatrick }
1121e5dd7070Spatrick
1122e5dd7070Spatrick ConsumedState
getState(const CXXBindTemporaryExpr * Tmp) const1123e5dd7070Spatrick ConsumedStateMap::getState(const CXXBindTemporaryExpr *Tmp) const {
1124e5dd7070Spatrick TmpMapType::const_iterator Entry = TmpMap.find(Tmp);
1125e5dd7070Spatrick
1126e5dd7070Spatrick if (Entry != TmpMap.end())
1127e5dd7070Spatrick return Entry->second;
1128e5dd7070Spatrick
1129e5dd7070Spatrick return CS_None;
1130e5dd7070Spatrick }
1131e5dd7070Spatrick
intersect(const ConsumedStateMap & Other)1132e5dd7070Spatrick void ConsumedStateMap::intersect(const ConsumedStateMap &Other) {
1133e5dd7070Spatrick ConsumedState LocalState;
1134e5dd7070Spatrick
1135e5dd7070Spatrick if (this->From && this->From == Other.From && !Other.Reachable) {
1136e5dd7070Spatrick this->markUnreachable();
1137e5dd7070Spatrick return;
1138e5dd7070Spatrick }
1139e5dd7070Spatrick
1140e5dd7070Spatrick for (const auto &DM : Other.VarMap) {
1141e5dd7070Spatrick LocalState = this->getState(DM.first);
1142e5dd7070Spatrick
1143e5dd7070Spatrick if (LocalState == CS_None)
1144e5dd7070Spatrick continue;
1145e5dd7070Spatrick
1146e5dd7070Spatrick if (LocalState != DM.second)
1147e5dd7070Spatrick VarMap[DM.first] = CS_Unknown;
1148e5dd7070Spatrick }
1149e5dd7070Spatrick }
1150e5dd7070Spatrick
intersectAtLoopHead(const CFGBlock * LoopHead,const CFGBlock * LoopBack,const ConsumedStateMap * LoopBackStates,ConsumedWarningsHandlerBase & WarningsHandler)1151e5dd7070Spatrick void ConsumedStateMap::intersectAtLoopHead(const CFGBlock *LoopHead,
1152e5dd7070Spatrick const CFGBlock *LoopBack, const ConsumedStateMap *LoopBackStates,
1153e5dd7070Spatrick ConsumedWarningsHandlerBase &WarningsHandler) {
1154e5dd7070Spatrick
1155e5dd7070Spatrick ConsumedState LocalState;
1156e5dd7070Spatrick SourceLocation BlameLoc = getLastStmtLoc(LoopBack);
1157e5dd7070Spatrick
1158e5dd7070Spatrick for (const auto &DM : LoopBackStates->VarMap) {
1159e5dd7070Spatrick LocalState = this->getState(DM.first);
1160e5dd7070Spatrick
1161e5dd7070Spatrick if (LocalState == CS_None)
1162e5dd7070Spatrick continue;
1163e5dd7070Spatrick
1164e5dd7070Spatrick if (LocalState != DM.second) {
1165e5dd7070Spatrick VarMap[DM.first] = CS_Unknown;
1166e5dd7070Spatrick WarningsHandler.warnLoopStateMismatch(BlameLoc,
1167e5dd7070Spatrick DM.first->getNameAsString());
1168e5dd7070Spatrick }
1169e5dd7070Spatrick }
1170e5dd7070Spatrick }
1171e5dd7070Spatrick
markUnreachable()1172e5dd7070Spatrick void ConsumedStateMap::markUnreachable() {
1173e5dd7070Spatrick this->Reachable = false;
1174e5dd7070Spatrick VarMap.clear();
1175e5dd7070Spatrick TmpMap.clear();
1176e5dd7070Spatrick }
1177e5dd7070Spatrick
setState(const VarDecl * Var,ConsumedState State)1178e5dd7070Spatrick void ConsumedStateMap::setState(const VarDecl *Var, ConsumedState State) {
1179e5dd7070Spatrick VarMap[Var] = State;
1180e5dd7070Spatrick }
1181e5dd7070Spatrick
setState(const CXXBindTemporaryExpr * Tmp,ConsumedState State)1182e5dd7070Spatrick void ConsumedStateMap::setState(const CXXBindTemporaryExpr *Tmp,
1183e5dd7070Spatrick ConsumedState State) {
1184e5dd7070Spatrick TmpMap[Tmp] = State;
1185e5dd7070Spatrick }
1186e5dd7070Spatrick
remove(const CXXBindTemporaryExpr * Tmp)1187e5dd7070Spatrick void ConsumedStateMap::remove(const CXXBindTemporaryExpr *Tmp) {
1188e5dd7070Spatrick TmpMap.erase(Tmp);
1189e5dd7070Spatrick }
1190e5dd7070Spatrick
operator !=(const ConsumedStateMap * Other) const1191e5dd7070Spatrick bool ConsumedStateMap::operator!=(const ConsumedStateMap *Other) const {
1192e5dd7070Spatrick for (const auto &DM : Other->VarMap)
1193e5dd7070Spatrick if (this->getState(DM.first) != DM.second)
1194e5dd7070Spatrick return true;
1195e5dd7070Spatrick return false;
1196e5dd7070Spatrick }
1197e5dd7070Spatrick
determineExpectedReturnState(AnalysisDeclContext & AC,const FunctionDecl * D)1198e5dd7070Spatrick void ConsumedAnalyzer::determineExpectedReturnState(AnalysisDeclContext &AC,
1199e5dd7070Spatrick const FunctionDecl *D) {
1200e5dd7070Spatrick QualType ReturnType;
1201e5dd7070Spatrick if (const auto *Constructor = dyn_cast<CXXConstructorDecl>(D)) {
1202e5dd7070Spatrick ReturnType = Constructor->getThisType()->getPointeeType();
1203e5dd7070Spatrick } else
1204e5dd7070Spatrick ReturnType = D->getCallResultType();
1205e5dd7070Spatrick
1206e5dd7070Spatrick if (const ReturnTypestateAttr *RTSAttr = D->getAttr<ReturnTypestateAttr>()) {
1207e5dd7070Spatrick const CXXRecordDecl *RD = ReturnType->getAsCXXRecordDecl();
1208e5dd7070Spatrick if (!RD || !RD->hasAttr<ConsumableAttr>()) {
1209e5dd7070Spatrick // FIXME: This should be removed when template instantiation propagates
1210e5dd7070Spatrick // attributes at template specialization definition, not
1211e5dd7070Spatrick // declaration. When it is removed the test needs to be enabled
1212e5dd7070Spatrick // in SemaDeclAttr.cpp.
1213e5dd7070Spatrick WarningsHandler.warnReturnTypestateForUnconsumableType(
1214e5dd7070Spatrick RTSAttr->getLocation(), ReturnType.getAsString());
1215e5dd7070Spatrick ExpectedReturnState = CS_None;
1216e5dd7070Spatrick } else
1217e5dd7070Spatrick ExpectedReturnState = mapReturnTypestateAttrState(RTSAttr);
1218e5dd7070Spatrick } else if (isConsumableType(ReturnType)) {
1219e5dd7070Spatrick if (isAutoCastType(ReturnType)) // We can auto-cast the state to the
1220e5dd7070Spatrick ExpectedReturnState = CS_None; // expected state.
1221e5dd7070Spatrick else
1222e5dd7070Spatrick ExpectedReturnState = mapConsumableAttrState(ReturnType);
1223e5dd7070Spatrick }
1224e5dd7070Spatrick else
1225e5dd7070Spatrick ExpectedReturnState = CS_None;
1226e5dd7070Spatrick }
1227e5dd7070Spatrick
splitState(const CFGBlock * CurrBlock,const ConsumedStmtVisitor & Visitor)1228e5dd7070Spatrick bool ConsumedAnalyzer::splitState(const CFGBlock *CurrBlock,
1229e5dd7070Spatrick const ConsumedStmtVisitor &Visitor) {
1230e5dd7070Spatrick std::unique_ptr<ConsumedStateMap> FalseStates(
1231e5dd7070Spatrick new ConsumedStateMap(*CurrStates));
1232e5dd7070Spatrick PropagationInfo PInfo;
1233e5dd7070Spatrick
1234e5dd7070Spatrick if (const auto *IfNode =
1235e5dd7070Spatrick dyn_cast_or_null<IfStmt>(CurrBlock->getTerminator().getStmt())) {
1236e5dd7070Spatrick const Expr *Cond = IfNode->getCond();
1237e5dd7070Spatrick
1238e5dd7070Spatrick PInfo = Visitor.getInfo(Cond);
1239e5dd7070Spatrick if (!PInfo.isValid() && isa<BinaryOperator>(Cond))
1240e5dd7070Spatrick PInfo = Visitor.getInfo(cast<BinaryOperator>(Cond)->getRHS());
1241e5dd7070Spatrick
1242e5dd7070Spatrick if (PInfo.isVarTest()) {
1243e5dd7070Spatrick CurrStates->setSource(Cond);
1244e5dd7070Spatrick FalseStates->setSource(Cond);
1245e5dd7070Spatrick splitVarStateForIf(IfNode, PInfo.getVarTest(), CurrStates.get(),
1246e5dd7070Spatrick FalseStates.get());
1247e5dd7070Spatrick } else if (PInfo.isBinTest()) {
1248e5dd7070Spatrick CurrStates->setSource(PInfo.testSourceNode());
1249e5dd7070Spatrick FalseStates->setSource(PInfo.testSourceNode());
1250e5dd7070Spatrick splitVarStateForIfBinOp(PInfo, CurrStates.get(), FalseStates.get());
1251e5dd7070Spatrick } else {
1252e5dd7070Spatrick return false;
1253e5dd7070Spatrick }
1254e5dd7070Spatrick } else if (const auto *BinOp =
1255e5dd7070Spatrick dyn_cast_or_null<BinaryOperator>(CurrBlock->getTerminator().getStmt())) {
1256e5dd7070Spatrick PInfo = Visitor.getInfo(BinOp->getLHS());
1257e5dd7070Spatrick if (!PInfo.isVarTest()) {
1258e5dd7070Spatrick if ((BinOp = dyn_cast_or_null<BinaryOperator>(BinOp->getLHS()))) {
1259e5dd7070Spatrick PInfo = Visitor.getInfo(BinOp->getRHS());
1260e5dd7070Spatrick
1261e5dd7070Spatrick if (!PInfo.isVarTest())
1262e5dd7070Spatrick return false;
1263e5dd7070Spatrick } else {
1264e5dd7070Spatrick return false;
1265e5dd7070Spatrick }
1266e5dd7070Spatrick }
1267e5dd7070Spatrick
1268e5dd7070Spatrick CurrStates->setSource(BinOp);
1269e5dd7070Spatrick FalseStates->setSource(BinOp);
1270e5dd7070Spatrick
1271e5dd7070Spatrick const VarTestResult &Test = PInfo.getVarTest();
1272e5dd7070Spatrick ConsumedState VarState = CurrStates->getState(Test.Var);
1273e5dd7070Spatrick
1274e5dd7070Spatrick if (BinOp->getOpcode() == BO_LAnd) {
1275e5dd7070Spatrick if (VarState == CS_Unknown)
1276e5dd7070Spatrick CurrStates->setState(Test.Var, Test.TestsFor);
1277e5dd7070Spatrick else if (VarState == invertConsumedUnconsumed(Test.TestsFor))
1278e5dd7070Spatrick CurrStates->markUnreachable();
1279e5dd7070Spatrick
1280e5dd7070Spatrick } else if (BinOp->getOpcode() == BO_LOr) {
1281e5dd7070Spatrick if (VarState == CS_Unknown)
1282e5dd7070Spatrick FalseStates->setState(Test.Var,
1283e5dd7070Spatrick invertConsumedUnconsumed(Test.TestsFor));
1284e5dd7070Spatrick else if (VarState == Test.TestsFor)
1285e5dd7070Spatrick FalseStates->markUnreachable();
1286e5dd7070Spatrick }
1287e5dd7070Spatrick } else {
1288e5dd7070Spatrick return false;
1289e5dd7070Spatrick }
1290e5dd7070Spatrick
1291e5dd7070Spatrick CFGBlock::const_succ_iterator SI = CurrBlock->succ_begin();
1292e5dd7070Spatrick
1293e5dd7070Spatrick if (*SI)
1294e5dd7070Spatrick BlockInfo.addInfo(*SI, std::move(CurrStates));
1295e5dd7070Spatrick else
1296e5dd7070Spatrick CurrStates = nullptr;
1297e5dd7070Spatrick
1298e5dd7070Spatrick if (*++SI)
1299e5dd7070Spatrick BlockInfo.addInfo(*SI, std::move(FalseStates));
1300e5dd7070Spatrick
1301e5dd7070Spatrick return true;
1302e5dd7070Spatrick }
1303e5dd7070Spatrick
run(AnalysisDeclContext & AC)1304e5dd7070Spatrick void ConsumedAnalyzer::run(AnalysisDeclContext &AC) {
1305e5dd7070Spatrick const auto *D = dyn_cast_or_null<FunctionDecl>(AC.getDecl());
1306e5dd7070Spatrick if (!D)
1307e5dd7070Spatrick return;
1308e5dd7070Spatrick
1309e5dd7070Spatrick CFG *CFGraph = AC.getCFG();
1310e5dd7070Spatrick if (!CFGraph)
1311e5dd7070Spatrick return;
1312e5dd7070Spatrick
1313e5dd7070Spatrick determineExpectedReturnState(AC, D);
1314e5dd7070Spatrick
1315e5dd7070Spatrick PostOrderCFGView *SortedGraph = AC.getAnalysis<PostOrderCFGView>();
1316e5dd7070Spatrick // AC.getCFG()->viewCFG(LangOptions());
1317e5dd7070Spatrick
1318e5dd7070Spatrick BlockInfo = ConsumedBlockInfo(CFGraph->getNumBlockIDs(), SortedGraph);
1319e5dd7070Spatrick
1320e5dd7070Spatrick CurrStates = std::make_unique<ConsumedStateMap>();
1321e5dd7070Spatrick ConsumedStmtVisitor Visitor(*this, CurrStates.get());
1322e5dd7070Spatrick
1323e5dd7070Spatrick // Add all trackable parameters to the state map.
1324e5dd7070Spatrick for (const auto *PI : D->parameters())
1325e5dd7070Spatrick Visitor.VisitParmVarDecl(PI);
1326e5dd7070Spatrick
1327e5dd7070Spatrick // Visit all of the function's basic blocks.
1328e5dd7070Spatrick for (const auto *CurrBlock : *SortedGraph) {
1329e5dd7070Spatrick if (!CurrStates)
1330e5dd7070Spatrick CurrStates = BlockInfo.getInfo(CurrBlock);
1331e5dd7070Spatrick
1332e5dd7070Spatrick if (!CurrStates) {
1333e5dd7070Spatrick continue;
1334e5dd7070Spatrick } else if (!CurrStates->isReachable()) {
1335e5dd7070Spatrick CurrStates = nullptr;
1336e5dd7070Spatrick continue;
1337e5dd7070Spatrick }
1338e5dd7070Spatrick
1339e5dd7070Spatrick Visitor.reset(CurrStates.get());
1340e5dd7070Spatrick
1341e5dd7070Spatrick // Visit all of the basic block's statements.
1342e5dd7070Spatrick for (const auto &B : *CurrBlock) {
1343e5dd7070Spatrick switch (B.getKind()) {
1344e5dd7070Spatrick case CFGElement::Statement:
1345e5dd7070Spatrick Visitor.Visit(B.castAs<CFGStmt>().getStmt());
1346e5dd7070Spatrick break;
1347e5dd7070Spatrick
1348e5dd7070Spatrick case CFGElement::TemporaryDtor: {
1349e5dd7070Spatrick const CFGTemporaryDtor &DTor = B.castAs<CFGTemporaryDtor>();
1350e5dd7070Spatrick const CXXBindTemporaryExpr *BTE = DTor.getBindTemporaryExpr();
1351e5dd7070Spatrick
1352e5dd7070Spatrick Visitor.checkCallability(PropagationInfo(BTE),
1353e5dd7070Spatrick DTor.getDestructorDecl(AC.getASTContext()),
1354e5dd7070Spatrick BTE->getExprLoc());
1355e5dd7070Spatrick CurrStates->remove(BTE);
1356e5dd7070Spatrick break;
1357e5dd7070Spatrick }
1358e5dd7070Spatrick
1359e5dd7070Spatrick case CFGElement::AutomaticObjectDtor: {
1360e5dd7070Spatrick const CFGAutomaticObjDtor &DTor = B.castAs<CFGAutomaticObjDtor>();
1361e5dd7070Spatrick SourceLocation Loc = DTor.getTriggerStmt()->getEndLoc();
1362e5dd7070Spatrick const VarDecl *Var = DTor.getVarDecl();
1363e5dd7070Spatrick
1364e5dd7070Spatrick Visitor.checkCallability(PropagationInfo(Var),
1365e5dd7070Spatrick DTor.getDestructorDecl(AC.getASTContext()),
1366e5dd7070Spatrick Loc);
1367e5dd7070Spatrick break;
1368e5dd7070Spatrick }
1369e5dd7070Spatrick
1370e5dd7070Spatrick default:
1371e5dd7070Spatrick break;
1372e5dd7070Spatrick }
1373e5dd7070Spatrick }
1374e5dd7070Spatrick
1375e5dd7070Spatrick // TODO: Handle other forms of branching with precision, including while-
1376e5dd7070Spatrick // and for-loops. (Deferred)
1377e5dd7070Spatrick if (!splitState(CurrBlock, Visitor)) {
1378e5dd7070Spatrick CurrStates->setSource(nullptr);
1379e5dd7070Spatrick
1380e5dd7070Spatrick if (CurrBlock->succ_size() > 1 ||
1381e5dd7070Spatrick (CurrBlock->succ_size() == 1 &&
1382e5dd7070Spatrick (*CurrBlock->succ_begin())->pred_size() > 1)) {
1383e5dd7070Spatrick
1384e5dd7070Spatrick auto *RawState = CurrStates.get();
1385e5dd7070Spatrick
1386e5dd7070Spatrick for (CFGBlock::const_succ_iterator SI = CurrBlock->succ_begin(),
1387e5dd7070Spatrick SE = CurrBlock->succ_end(); SI != SE; ++SI) {
1388e5dd7070Spatrick if (*SI == nullptr) continue;
1389e5dd7070Spatrick
1390e5dd7070Spatrick if (BlockInfo.isBackEdge(CurrBlock, *SI)) {
1391e5dd7070Spatrick BlockInfo.borrowInfo(*SI)->intersectAtLoopHead(
1392e5dd7070Spatrick *SI, CurrBlock, RawState, WarningsHandler);
1393e5dd7070Spatrick
1394e5dd7070Spatrick if (BlockInfo.allBackEdgesVisited(CurrBlock, *SI))
1395e5dd7070Spatrick BlockInfo.discardInfo(*SI);
1396e5dd7070Spatrick } else {
1397e5dd7070Spatrick BlockInfo.addInfo(*SI, RawState, CurrStates);
1398e5dd7070Spatrick }
1399e5dd7070Spatrick }
1400e5dd7070Spatrick
1401e5dd7070Spatrick CurrStates = nullptr;
1402e5dd7070Spatrick }
1403e5dd7070Spatrick }
1404e5dd7070Spatrick
1405e5dd7070Spatrick if (CurrBlock == &AC.getCFG()->getExit() &&
1406e5dd7070Spatrick D->getCallResultType()->isVoidType())
1407e5dd7070Spatrick CurrStates->checkParamsForReturnTypestate(D->getLocation(),
1408e5dd7070Spatrick WarningsHandler);
1409e5dd7070Spatrick } // End of block iterator.
1410e5dd7070Spatrick
1411e5dd7070Spatrick // Delete the last existing state map.
1412e5dd7070Spatrick CurrStates = nullptr;
1413e5dd7070Spatrick
1414e5dd7070Spatrick WarningsHandler.emitDiagnostics();
1415e5dd7070Spatrick }
1416