xref: /llvm-project/clang/lib/Analysis/FlowSensitive/Models/UncheckedOptionalAccessModel.cpp (revision ffb4f4db73419d9eba81efe00aa0d498f01f6d27)
1b000b770SStanislav Gatev //===-- UncheckedOptionalAccessModel.cpp ------------------------*- C++ -*-===//
2b000b770SStanislav Gatev //
3b000b770SStanislav Gatev // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
4b000b770SStanislav Gatev // See https://llvm.org/LICENSE.txt for license information.
5b000b770SStanislav Gatev // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
6b000b770SStanislav Gatev //
7b000b770SStanislav Gatev //===----------------------------------------------------------------------===//
8b000b770SStanislav Gatev //
9b000b770SStanislav Gatev //  This file defines a dataflow analysis that detects unsafe uses of optional
10b000b770SStanislav Gatev //  values.
11b000b770SStanislav Gatev //
12b000b770SStanislav Gatev //===----------------------------------------------------------------------===//
13b000b770SStanislav Gatev 
14af98b0afSStanislav Gatev #include "clang/Analysis/FlowSensitive/Models/UncheckedOptionalAccessModel.h"
15af98b0afSStanislav Gatev #include "clang/AST/ASTContext.h"
167e63a0d4SYitzhak Mandelbaum #include "clang/AST/DeclCXX.h"
17af98b0afSStanislav Gatev #include "clang/AST/Expr.h"
18af98b0afSStanislav Gatev #include "clang/AST/ExprCXX.h"
19af98b0afSStanislav Gatev #include "clang/AST/Stmt.h"
20af98b0afSStanislav Gatev #include "clang/ASTMatchers/ASTMatchers.h"
2109b462efSYitzhak Mandelbaum #include "clang/ASTMatchers/ASTMatchersMacros.h"
227538b360SWei Yi Tee #include "clang/Analysis/CFG.h"
237538b360SWei Yi Tee #include "clang/Analysis/FlowSensitive/CFGMatchSwitch.h"
24af98b0afSStanislav Gatev #include "clang/Analysis/FlowSensitive/DataflowEnvironment.h"
25cf1f978dSSam Estep #include "clang/Analysis/FlowSensitive/NoopLattice.h"
260086a355SYitzhak Mandelbaum #include "clang/Analysis/FlowSensitive/StorageLocation.h"
27af98b0afSStanislav Gatev #include "clang/Analysis/FlowSensitive/Value.h"
2858fe7f96SSam Estep #include "clang/Basic/SourceLocation.h"
29af98b0afSStanislav Gatev #include "llvm/ADT/StringRef.h"
30af98b0afSStanislav Gatev #include "llvm/Support/Casting.h"
31d34fbf2dSYitzhak Mandelbaum #include "llvm/Support/ErrorHandling.h"
32af98b0afSStanislav Gatev #include <cassert>
339e0fc676SStanislav Gatev #include <memory>
34a1580d7bSKazu Hirata #include <optional>
359e0fc676SStanislav Gatev #include <utility>
3658fe7f96SSam Estep #include <vector>
37af98b0afSStanislav Gatev 
38af98b0afSStanislav Gatev namespace clang {
39af98b0afSStanislav Gatev namespace dataflow {
4009b462efSYitzhak Mandelbaum 
4109b462efSYitzhak Mandelbaum static bool isTopLevelNamespaceWithName(const NamespaceDecl &NS,
4209b462efSYitzhak Mandelbaum                                         llvm::StringRef Name) {
4309b462efSYitzhak Mandelbaum   return NS.getDeclName().isIdentifier() && NS.getName() == Name &&
4409b462efSYitzhak Mandelbaum          NS.getParent() != nullptr && NS.getParent()->isTranslationUnit();
4509b462efSYitzhak Mandelbaum }
4609b462efSYitzhak Mandelbaum 
4709b462efSYitzhak Mandelbaum static bool hasOptionalClassName(const CXXRecordDecl &RD) {
4809b462efSYitzhak Mandelbaum   if (!RD.getDeclName().isIdentifier())
4909b462efSYitzhak Mandelbaum     return false;
5009b462efSYitzhak Mandelbaum 
5109b462efSYitzhak Mandelbaum   if (RD.getName() == "optional") {
5209b462efSYitzhak Mandelbaum     if (const auto *N = dyn_cast_or_null<NamespaceDecl>(RD.getDeclContext()))
5309b462efSYitzhak Mandelbaum       return N->isStdNamespace() || isTopLevelNamespaceWithName(*N, "absl");
5409b462efSYitzhak Mandelbaum     return false;
5509b462efSYitzhak Mandelbaum   }
5609b462efSYitzhak Mandelbaum 
5709b462efSYitzhak Mandelbaum   if (RD.getName() == "Optional") {
5809b462efSYitzhak Mandelbaum     // Check whether namespace is "::base".
5909b462efSYitzhak Mandelbaum     const auto *N = dyn_cast_or_null<NamespaceDecl>(RD.getDeclContext());
6009b462efSYitzhak Mandelbaum     return N != nullptr && isTopLevelNamespaceWithName(*N, "base");
6109b462efSYitzhak Mandelbaum   }
6209b462efSYitzhak Mandelbaum 
6309b462efSYitzhak Mandelbaum   return false;
6409b462efSYitzhak Mandelbaum }
6509b462efSYitzhak Mandelbaum 
66af98b0afSStanislav Gatev namespace {
67af98b0afSStanislav Gatev 
68af98b0afSStanislav Gatev using namespace ::clang::ast_matchers;
69cf1f978dSSam Estep using LatticeTransferState = TransferState<NoopLattice>;
70af98b0afSStanislav Gatev 
7109b462efSYitzhak Mandelbaum AST_MATCHER(CXXRecordDecl, hasOptionalClassNameMatcher) {
7209b462efSYitzhak Mandelbaum   return hasOptionalClassName(Node);
7309b462efSYitzhak Mandelbaum }
7409b462efSYitzhak Mandelbaum 
757e63a0d4SYitzhak Mandelbaum DeclarationMatcher optionalClass() {
76af98b0afSStanislav Gatev   return classTemplateSpecializationDecl(
7709b462efSYitzhak Mandelbaum       hasOptionalClassNameMatcher(),
78af98b0afSStanislav Gatev       hasTemplateArgument(0, refersToType(type().bind("T"))));
79af98b0afSStanislav Gatev }
80af98b0afSStanislav Gatev 
816adfc64eSYitzhak Mandelbaum auto optionalOrAliasType() {
8265e710c3SStanislav Gatev   return hasUnqualifiedDesugaredType(
8365e710c3SStanislav Gatev       recordType(hasDeclaration(optionalClass())));
8465e710c3SStanislav Gatev }
8565e710c3SStanislav Gatev 
866adfc64eSYitzhak Mandelbaum /// Matches any of the spellings of the optional types and sugar, aliases, etc.
876adfc64eSYitzhak Mandelbaum auto hasOptionalType() { return hasType(optionalOrAliasType()); }
886adfc64eSYitzhak Mandelbaum 
89a184a0d8SYitzhak Mandelbaum auto isOptionalMemberCallWithName(
90a184a0d8SYitzhak Mandelbaum     llvm::StringRef MemberName,
916ad0788cSKazu Hirata     const std::optional<StatementMatcher> &Ignorable = std::nullopt) {
92a184a0d8SYitzhak Mandelbaum   auto Exception = unless(Ignorable ? expr(anyOf(*Ignorable, cxxThisExpr()))
93a184a0d8SYitzhak Mandelbaum                                     : cxxThisExpr());
94af98b0afSStanislav Gatev   return cxxMemberCallExpr(
9509b462efSYitzhak Mandelbaum       on(expr(Exception,
9609b462efSYitzhak Mandelbaum               anyOf(hasOptionalType(),
9709b462efSYitzhak Mandelbaum                     hasType(pointerType(pointee(optionalOrAliasType())))))),
9809b462efSYitzhak Mandelbaum       callee(cxxMethodDecl(hasName(MemberName))));
99af98b0afSStanislav Gatev }
100af98b0afSStanislav Gatev 
101a184a0d8SYitzhak Mandelbaum auto isOptionalOperatorCallWithName(
102a184a0d8SYitzhak Mandelbaum     llvm::StringRef operator_name,
1036ad0788cSKazu Hirata     const std::optional<StatementMatcher> &Ignorable = std::nullopt) {
104a184a0d8SYitzhak Mandelbaum   return cxxOperatorCallExpr(
105a184a0d8SYitzhak Mandelbaum       hasOverloadedOperatorName(operator_name),
106a184a0d8SYitzhak Mandelbaum       callee(cxxMethodDecl(ofClass(optionalClass()))),
107a184a0d8SYitzhak Mandelbaum       Ignorable ? callExpr(unless(hasArgument(0, *Ignorable))) : callExpr());
108af98b0afSStanislav Gatev }
109af98b0afSStanislav Gatev 
110092a530cSStanislav Gatev auto isMakeOptionalCall() {
1119e0fc676SStanislav Gatev   return callExpr(
1129e0fc676SStanislav Gatev       callee(functionDecl(hasAnyName(
1139e0fc676SStanislav Gatev           "std::make_optional", "base::make_optional", "absl::make_optional"))),
1149e0fc676SStanislav Gatev       hasOptionalType());
1159e0fc676SStanislav Gatev }
1169e0fc676SStanislav Gatev 
117390029beSYitzhak Mandelbaum auto nulloptTypeDecl() {
118390029beSYitzhak Mandelbaum   return namedDecl(
119390029beSYitzhak Mandelbaum       hasAnyName("std::nullopt_t", "absl::nullopt_t", "base::nullopt_t"));
120092a530cSStanislav Gatev }
121092a530cSStanislav Gatev 
122390029beSYitzhak Mandelbaum auto hasNulloptType() { return hasType(nulloptTypeDecl()); }
123390029beSYitzhak Mandelbaum 
124390029beSYitzhak Mandelbaum // `optional` or `nullopt_t`
125390029beSYitzhak Mandelbaum auto hasAnyOptionalType() {
126390029beSYitzhak Mandelbaum   return hasType(hasUnqualifiedDesugaredType(
127390029beSYitzhak Mandelbaum       recordType(hasDeclaration(anyOf(nulloptTypeDecl(), optionalClass())))));
128390029beSYitzhak Mandelbaum }
129390029beSYitzhak Mandelbaum 
130092a530cSStanislav Gatev auto inPlaceClass() {
131092a530cSStanislav Gatev   return recordDecl(
132092a530cSStanislav Gatev       hasAnyName("std::in_place_t", "absl::in_place_t", "base::in_place_t"));
133092a530cSStanislav Gatev }
134092a530cSStanislav Gatev 
135092a530cSStanislav Gatev auto isOptionalNulloptConstructor() {
1360086a355SYitzhak Mandelbaum   return cxxConstructExpr(
1370086a355SYitzhak Mandelbaum       hasOptionalType(),
1380086a355SYitzhak Mandelbaum       hasDeclaration(cxxConstructorDecl(parameterCountIs(1),
1390086a355SYitzhak Mandelbaum                                         hasParameter(0, hasNulloptType()))));
140092a530cSStanislav Gatev }
141092a530cSStanislav Gatev 
142092a530cSStanislav Gatev auto isOptionalInPlaceConstructor() {
143092a530cSStanislav Gatev   return cxxConstructExpr(hasOptionalType(),
144092a530cSStanislav Gatev                           hasArgument(0, hasType(inPlaceClass())));
145092a530cSStanislav Gatev }
146092a530cSStanislav Gatev 
147092a530cSStanislav Gatev auto isOptionalValueOrConversionConstructor() {
148092a530cSStanislav Gatev   return cxxConstructExpr(
149092a530cSStanislav Gatev       hasOptionalType(),
150092a530cSStanislav Gatev       unless(hasDeclaration(
151092a530cSStanislav Gatev           cxxConstructorDecl(anyOf(isCopyConstructor(), isMoveConstructor())))),
152092a530cSStanislav Gatev       argumentCountIs(1), hasArgument(0, unless(hasNulloptType())));
153092a530cSStanislav Gatev }
154092a530cSStanislav Gatev 
155b000b770SStanislav Gatev auto isOptionalValueOrConversionAssignment() {
156b000b770SStanislav Gatev   return cxxOperatorCallExpr(
157b000b770SStanislav Gatev       hasOverloadedOperatorName("="),
158b000b770SStanislav Gatev       callee(cxxMethodDecl(ofClass(optionalClass()))),
159b000b770SStanislav Gatev       unless(hasDeclaration(cxxMethodDecl(
160b000b770SStanislav Gatev           anyOf(isCopyAssignmentOperator(), isMoveAssignmentOperator())))),
161b000b770SStanislav Gatev       argumentCountIs(2), hasArgument(1, unless(hasNulloptType())));
162b000b770SStanislav Gatev }
163b000b770SStanislav Gatev 
164390029beSYitzhak Mandelbaum auto isNulloptConstructor() {
165390029beSYitzhak Mandelbaum   return cxxConstructExpr(hasNulloptType(), argumentCountIs(1),
166390029beSYitzhak Mandelbaum                           hasArgument(0, hasNulloptType()));
167390029beSYitzhak Mandelbaum }
168390029beSYitzhak Mandelbaum 
169b000b770SStanislav Gatev auto isOptionalNulloptAssignment() {
170b000b770SStanislav Gatev   return cxxOperatorCallExpr(hasOverloadedOperatorName("="),
171b000b770SStanislav Gatev                              callee(cxxMethodDecl(ofClass(optionalClass()))),
172b000b770SStanislav Gatev                              argumentCountIs(2),
173b000b770SStanislav Gatev                              hasArgument(1, hasNulloptType()));
174b000b770SStanislav Gatev }
175b000b770SStanislav Gatev 
1762ddd57aeSStanislav Gatev auto isStdSwapCall() {
1772ddd57aeSStanislav Gatev   return callExpr(callee(functionDecl(hasName("std::swap"))),
1782ddd57aeSStanislav Gatev                   argumentCountIs(2), hasArgument(0, hasOptionalType()),
1792ddd57aeSStanislav Gatev                   hasArgument(1, hasOptionalType()));
1802ddd57aeSStanislav Gatev }
1812ddd57aeSStanislav Gatev 
18225956d55SAMS21 auto isStdForwardCall() {
18325956d55SAMS21   return callExpr(callee(functionDecl(hasName("std::forward"))),
18425956d55SAMS21                   argumentCountIs(1), hasArgument(0, hasOptionalType()));
18525956d55SAMS21 }
18625956d55SAMS21 
1877f076004SYitzhak Mandelbaum constexpr llvm::StringLiteral ValueOrCallID = "ValueOrCall";
1887f076004SYitzhak Mandelbaum 
1897f076004SYitzhak Mandelbaum auto isValueOrStringEmptyCall() {
1907f076004SYitzhak Mandelbaum   // `opt.value_or("").empty()`
1917f076004SYitzhak Mandelbaum   return cxxMemberCallExpr(
1927f076004SYitzhak Mandelbaum       callee(cxxMethodDecl(hasName("empty"))),
1937f076004SYitzhak Mandelbaum       onImplicitObjectArgument(ignoringImplicit(
1947f076004SYitzhak Mandelbaum           cxxMemberCallExpr(on(expr(unless(cxxThisExpr()))),
1957f076004SYitzhak Mandelbaum                             callee(cxxMethodDecl(hasName("value_or"),
1967f076004SYitzhak Mandelbaum                                                  ofClass(optionalClass()))),
1977f076004SYitzhak Mandelbaum                             hasArgument(0, stringLiteral(hasSize(0))))
1987f076004SYitzhak Mandelbaum               .bind(ValueOrCallID))));
1997f076004SYitzhak Mandelbaum }
2007f076004SYitzhak Mandelbaum 
2017f076004SYitzhak Mandelbaum auto isValueOrNotEqX() {
2027f076004SYitzhak Mandelbaum   auto ComparesToSame = [](ast_matchers::internal::Matcher<Stmt> Arg) {
2037f076004SYitzhak Mandelbaum     return hasOperands(
2047f076004SYitzhak Mandelbaum         ignoringImplicit(
2057f076004SYitzhak Mandelbaum             cxxMemberCallExpr(on(expr(unless(cxxThisExpr()))),
2067f076004SYitzhak Mandelbaum                               callee(cxxMethodDecl(hasName("value_or"),
2077f076004SYitzhak Mandelbaum                                                    ofClass(optionalClass()))),
2087f076004SYitzhak Mandelbaum                               hasArgument(0, Arg))
2097f076004SYitzhak Mandelbaum                 .bind(ValueOrCallID)),
2107f076004SYitzhak Mandelbaum         ignoringImplicit(Arg));
2117f076004SYitzhak Mandelbaum   };
2127f076004SYitzhak Mandelbaum 
2137f076004SYitzhak Mandelbaum   // `opt.value_or(X) != X`, for X is `nullptr`, `""`, or `0`. Ideally, we'd
2147f076004SYitzhak Mandelbaum   // support this pattern for any expression, but the AST does not have a
2157f076004SYitzhak Mandelbaum   // generic expression comparison facility, so we specialize to common cases
2167f076004SYitzhak Mandelbaum   // seen in practice.  FIXME: define a matcher that compares values across
2177f076004SYitzhak Mandelbaum   // nodes, which would let us generalize this to any `X`.
2187f076004SYitzhak Mandelbaum   return binaryOperation(hasOperatorName("!="),
2197f076004SYitzhak Mandelbaum                          anyOf(ComparesToSame(cxxNullPtrLiteralExpr()),
2207f076004SYitzhak Mandelbaum                                ComparesToSame(stringLiteral(hasSize(0))),
2217f076004SYitzhak Mandelbaum                                ComparesToSame(integerLiteral(equals(0)))));
2227f076004SYitzhak Mandelbaum }
2237f076004SYitzhak Mandelbaum 
22465e710c3SStanislav Gatev auto isCallReturningOptional() {
225cd0d5261SSam Estep   return callExpr(hasType(qualType(anyOf(
226cd0d5261SSam Estep       optionalOrAliasType(), referenceType(pointee(optionalOrAliasType()))))));
22765e710c3SStanislav Gatev }
22865e710c3SStanislav Gatev 
229390029beSYitzhak Mandelbaum template <typename L, typename R>
230390029beSYitzhak Mandelbaum auto isComparisonOperatorCall(L lhs_arg_matcher, R rhs_arg_matcher) {
231390029beSYitzhak Mandelbaum   return cxxOperatorCallExpr(
232390029beSYitzhak Mandelbaum       anyOf(hasOverloadedOperatorName("=="), hasOverloadedOperatorName("!=")),
233390029beSYitzhak Mandelbaum       argumentCountIs(2), hasArgument(0, lhs_arg_matcher),
234390029beSYitzhak Mandelbaum       hasArgument(1, rhs_arg_matcher));
235390029beSYitzhak Mandelbaum }
236390029beSYitzhak Mandelbaum 
237*ffb4f4dbSDmitri Gribenko /// Ensures that `Expr` is mapped to a `BoolValue` and returns it.
238390029beSYitzhak Mandelbaum BoolValue &forceBoolValue(Environment &Env, const Expr &Expr) {
239390029beSYitzhak Mandelbaum   auto *Value = cast_or_null<BoolValue>(Env.getValue(Expr, SkipPast::None));
240390029beSYitzhak Mandelbaum   if (Value != nullptr)
241390029beSYitzhak Mandelbaum     return *Value;
242390029beSYitzhak Mandelbaum 
243390029beSYitzhak Mandelbaum   auto &Loc = Env.createStorageLocation(Expr);
244390029beSYitzhak Mandelbaum   Value = &Env.makeAtomicBoolValue();
245390029beSYitzhak Mandelbaum   Env.setValue(Loc, *Value);
246390029beSYitzhak Mandelbaum   Env.setStorageLocation(Expr, Loc);
247390029beSYitzhak Mandelbaum   return *Value;
248390029beSYitzhak Mandelbaum }
249390029beSYitzhak Mandelbaum 
2508fcdd625SStanislav Gatev /// Sets `HasValueVal` as the symbolic value that represents the "has_value"
2518fcdd625SStanislav Gatev /// property of the optional value `OptionalVal`.
2528fcdd625SStanislav Gatev void setHasValue(Value &OptionalVal, BoolValue &HasValueVal) {
2538fcdd625SStanislav Gatev   OptionalVal.setProperty("has_value", HasValueVal);
2548fcdd625SStanislav Gatev }
2558fcdd625SStanislav Gatev 
2569e0fc676SStanislav Gatev /// Creates a symbolic value for an `optional` value using `HasValueVal` as the
2579e0fc676SStanislav Gatev /// symbolic value of its "has_value" property.
2589e0fc676SStanislav Gatev StructValue &createOptionalValue(Environment &Env, BoolValue &HasValueVal) {
259745a957fSMartin Braenne   auto &OptionalVal = Env.create<StructValue>();
260745a957fSMartin Braenne   setHasValue(OptionalVal, HasValueVal);
261745a957fSMartin Braenne   return OptionalVal;
2629e0fc676SStanislav Gatev }
2639e0fc676SStanislav Gatev 
264af98b0afSStanislav Gatev /// Returns the symbolic value that represents the "has_value" property of the
26549ed5bf5SWei Yi Tee /// optional value `OptionalVal`. Returns null if `OptionalVal` is null.
266dd38caf3SYitzhak Mandelbaum BoolValue *getHasValue(Environment &Env, Value *OptionalVal) {
267dd38caf3SYitzhak Mandelbaum   if (OptionalVal != nullptr) {
268dd38caf3SYitzhak Mandelbaum     auto *HasValueVal =
269dd38caf3SYitzhak Mandelbaum         cast_or_null<BoolValue>(OptionalVal->getProperty("has_value"));
270dd38caf3SYitzhak Mandelbaum     if (HasValueVal == nullptr) {
271dd38caf3SYitzhak Mandelbaum       HasValueVal = &Env.makeAtomicBoolValue();
272dd38caf3SYitzhak Mandelbaum       OptionalVal->setProperty("has_value", *HasValueVal);
273dd38caf3SYitzhak Mandelbaum     }
274dd38caf3SYitzhak Mandelbaum     return HasValueVal;
275af98b0afSStanislav Gatev   }
276af98b0afSStanislav Gatev   return nullptr;
277af98b0afSStanislav Gatev }
278af98b0afSStanislav Gatev 
279092a530cSStanislav Gatev /// Returns true if and only if `Type` is an optional type.
280c0725865SYitzhak Mandelbaum bool isOptionalType(QualType Type) {
281092a530cSStanislav Gatev   if (!Type->isRecordType())
282092a530cSStanislav Gatev     return false;
283cd22e0dcSYitzhak Mandelbaum   const CXXRecordDecl *D = Type->getAsCXXRecordDecl();
28409b462efSYitzhak Mandelbaum   return D != nullptr && hasOptionalClassName(*D);
285092a530cSStanislav Gatev }
286092a530cSStanislav Gatev 
287092a530cSStanislav Gatev /// Returns the number of optional wrappers in `Type`.
288092a530cSStanislav Gatev ///
289092a530cSStanislav Gatev /// For example, if `Type` is `optional<optional<int>>`, the result of this
290092a530cSStanislav Gatev /// function will be 2.
291092a530cSStanislav Gatev int countOptionalWrappers(const ASTContext &ASTCtx, QualType Type) {
292c0725865SYitzhak Mandelbaum   if (!isOptionalType(Type))
293092a530cSStanislav Gatev     return 0;
294092a530cSStanislav Gatev   return 1 + countOptionalWrappers(
295092a530cSStanislav Gatev                  ASTCtx,
296092a530cSStanislav Gatev                  cast<ClassTemplateSpecializationDecl>(Type->getAsRecordDecl())
297092a530cSStanislav Gatev                      ->getTemplateArgs()
298092a530cSStanislav Gatev                      .get(0)
299092a530cSStanislav Gatev                      .getAsType()
300092a530cSStanislav Gatev                      .getDesugaredType(ASTCtx));
301092a530cSStanislav Gatev }
302092a530cSStanislav Gatev 
303dd38caf3SYitzhak Mandelbaum /// Tries to initialize the `optional`'s value (that is, contents), and return
304dd38caf3SYitzhak Mandelbaum /// its location. Returns nullptr if the value can't be represented.
305dd38caf3SYitzhak Mandelbaum StorageLocation *maybeInitializeOptionalValueMember(QualType Q,
306dd38caf3SYitzhak Mandelbaum                                                     Value &OptionalVal,
307dd38caf3SYitzhak Mandelbaum                                                     Environment &Env) {
308dd38caf3SYitzhak Mandelbaum   // The "value" property represents a synthetic field. As such, it needs
309dd38caf3SYitzhak Mandelbaum   // `StorageLocation`, like normal fields (and other variables). So, we model
310dd38caf3SYitzhak Mandelbaum   // it with a `ReferenceValue`, since that includes a storage location.  Once
311dd38caf3SYitzhak Mandelbaum   // the property is set, it will be shared by all environments that access the
312dd38caf3SYitzhak Mandelbaum   // `Value` representing the optional (here, `OptionalVal`).
313dd38caf3SYitzhak Mandelbaum   if (auto *ValueProp = OptionalVal.getProperty("value")) {
314dd38caf3SYitzhak Mandelbaum     auto *ValueRef = clang::cast<ReferenceValue>(ValueProp);
31597d69cdaSWei Yi Tee     auto &ValueLoc = ValueRef->getReferentLoc();
316dd38caf3SYitzhak Mandelbaum     if (Env.getValue(ValueLoc) == nullptr) {
317dd38caf3SYitzhak Mandelbaum       // The property was previously set, but the value has been lost. This can
318dd38caf3SYitzhak Mandelbaum       // happen, for example, because of an environment merge (where the two
319dd38caf3SYitzhak Mandelbaum       // environments mapped the property to different values, which resulted in
320dd38caf3SYitzhak Mandelbaum       // them both being discarded), or when two blocks in the CFG, with neither
321dd38caf3SYitzhak Mandelbaum       // a dominator of the other, visit the same optional value, or even when a
322dd38caf3SYitzhak Mandelbaum       // block is revisited during testing to collect per-statement state.
323dd38caf3SYitzhak Mandelbaum       // FIXME: This situation means that the optional contents are not shared
324dd38caf3SYitzhak Mandelbaum       // between branches and the like. Practically, this lack of sharing
325dd38caf3SYitzhak Mandelbaum       // reduces the precision of the model when the contents are relevant to
326dd38caf3SYitzhak Mandelbaum       // the check, like another optional or a boolean that influences control
327dd38caf3SYitzhak Mandelbaum       // flow.
328dd38caf3SYitzhak Mandelbaum       auto *ValueVal = Env.createValue(ValueLoc.getType());
329dd38caf3SYitzhak Mandelbaum       if (ValueVal == nullptr)
330dd38caf3SYitzhak Mandelbaum         return nullptr;
331dd38caf3SYitzhak Mandelbaum       Env.setValue(ValueLoc, *ValueVal);
332dd38caf3SYitzhak Mandelbaum     }
333dd38caf3SYitzhak Mandelbaum     return &ValueLoc;
334dd38caf3SYitzhak Mandelbaum   }
335dd38caf3SYitzhak Mandelbaum 
336c849843cSMartin Braenne   auto Ty = Q.getNonReferenceType();
337dd38caf3SYitzhak Mandelbaum   auto *ValueVal = Env.createValue(Ty);
338dd38caf3SYitzhak Mandelbaum   if (ValueVal == nullptr)
339dd38caf3SYitzhak Mandelbaum     return nullptr;
340dd38caf3SYitzhak Mandelbaum   auto &ValueLoc = Env.createStorageLocation(Ty);
341dd38caf3SYitzhak Mandelbaum   Env.setValue(ValueLoc, *ValueVal);
342745a957fSMartin Braenne   auto &ValueRef = Env.create<ReferenceValue>(ValueLoc);
343745a957fSMartin Braenne   OptionalVal.setProperty("value", ValueRef);
344dd38caf3SYitzhak Mandelbaum   return &ValueLoc;
345dd38caf3SYitzhak Mandelbaum }
346dd38caf3SYitzhak Mandelbaum 
347092a530cSStanislav Gatev void initializeOptionalReference(const Expr *OptionalExpr,
348092a530cSStanislav Gatev                                  const MatchFinder::MatchResult &,
349af98b0afSStanislav Gatev                                  LatticeTransferState &State) {
35049ed5bf5SWei Yi Tee   if (auto *OptionalVal =
35149ed5bf5SWei Yi Tee           State.Env.getValue(*OptionalExpr, SkipPast::Reference)) {
352af98b0afSStanislav Gatev     if (OptionalVal->getProperty("has_value") == nullptr) {
3538fcdd625SStanislav Gatev       setHasValue(*OptionalVal, State.Env.makeAtomicBoolValue());
354af98b0afSStanislav Gatev     }
355af98b0afSStanislav Gatev   }
356af98b0afSStanislav Gatev }
357af98b0afSStanislav Gatev 
3588fcdd625SStanislav Gatev /// Returns true if and only if `OptionalVal` is initialized and known to be
359*ffb4f4dbSDmitri Gribenko /// empty in `Env`.
3608fcdd625SStanislav Gatev bool isEmptyOptional(const Value &OptionalVal, const Environment &Env) {
3618fcdd625SStanislav Gatev   auto *HasValueVal =
3628fcdd625SStanislav Gatev       cast_or_null<BoolValue>(OptionalVal.getProperty("has_value"));
3638fcdd625SStanislav Gatev   return HasValueVal != nullptr &&
3648fcdd625SStanislav Gatev          Env.flowConditionImplies(Env.makeNot(*HasValueVal));
3658fcdd625SStanislav Gatev }
3668fcdd625SStanislav Gatev 
3678fcdd625SStanislav Gatev /// Returns true if and only if `OptionalVal` is initialized and known to be
368*ffb4f4dbSDmitri Gribenko /// non-empty in `Env`.
3698fcdd625SStanislav Gatev bool isNonEmptyOptional(const Value &OptionalVal, const Environment &Env) {
3708fcdd625SStanislav Gatev   auto *HasValueVal =
3718fcdd625SStanislav Gatev       cast_or_null<BoolValue>(OptionalVal.getProperty("has_value"));
3728fcdd625SStanislav Gatev   return HasValueVal != nullptr && Env.flowConditionImplies(*HasValueVal);
3738fcdd625SStanislav Gatev }
3748fcdd625SStanislav Gatev 
37548bc7150SMartin Braenne StorageLocation *maybeSkipPointer(StorageLocation *Loc,
37648bc7150SMartin Braenne                                   const Environment &Env) {
37748bc7150SMartin Braenne   if (Loc == nullptr)
37848bc7150SMartin Braenne     return nullptr;
37948bc7150SMartin Braenne   if (auto *Val = dyn_cast_or_null<PointerValue>(Env.getValue(*Loc)))
38048bc7150SMartin Braenne     return &Val->getPointeeLoc();
38148bc7150SMartin Braenne   return Loc;
38248bc7150SMartin Braenne }
38348bc7150SMartin Braenne 
38448bc7150SMartin Braenne Value *getValueBehindPossiblePointer(const Expr &E, const Environment &Env) {
38548bc7150SMartin Braenne   Value *Val = Env.getValue(E, SkipPast::Reference);
38648bc7150SMartin Braenne   if (auto *PointerVal = dyn_cast_or_null<PointerValue>(Val))
38748bc7150SMartin Braenne     return Env.getValue(PointerVal->getPointeeLoc());
38848bc7150SMartin Braenne   return Val;
38948bc7150SMartin Braenne }
39048bc7150SMartin Braenne 
391092a530cSStanislav Gatev void transferUnwrapCall(const Expr *UnwrapExpr, const Expr *ObjectExpr,
392af98b0afSStanislav Gatev                         LatticeTransferState &State) {
39349ed5bf5SWei Yi Tee   if (auto *OptionalVal =
39448bc7150SMartin Braenne           getValueBehindPossiblePointer(*ObjectExpr, State.Env)) {
395dd38caf3SYitzhak Mandelbaum     if (State.Env.getStorageLocation(*UnwrapExpr, SkipPast::None) == nullptr)
396dd38caf3SYitzhak Mandelbaum       if (auto *Loc = maybeInitializeOptionalValueMember(
397dd38caf3SYitzhak Mandelbaum               UnwrapExpr->getType(), *OptionalVal, State.Env))
398dd38caf3SYitzhak Mandelbaum         State.Env.setStorageLocation(*UnwrapExpr, *Loc);
399af98b0afSStanislav Gatev   }
400dd38caf3SYitzhak Mandelbaum }
401af98b0afSStanislav Gatev 
402092a530cSStanislav Gatev void transferMakeOptionalCall(const CallExpr *E,
403092a530cSStanislav Gatev                               const MatchFinder::MatchResult &,
404092a530cSStanislav Gatev                               LatticeTransferState &State) {
4059e0fc676SStanislav Gatev   auto &Loc = State.Env.createStorageLocation(*E);
4069e0fc676SStanislav Gatev   State.Env.setStorageLocation(*E, Loc);
4079e0fc676SStanislav Gatev   State.Env.setValue(
4089e0fc676SStanislav Gatev       Loc, createOptionalValue(State.Env, State.Env.getBoolLiteralValue(true)));
4099e0fc676SStanislav Gatev }
4109e0fc676SStanislav Gatev 
411092a530cSStanislav Gatev void transferOptionalHasValueCall(const CXXMemberCallExpr *CallExpr,
412092a530cSStanislav Gatev                                   const MatchFinder::MatchResult &,
413af98b0afSStanislav Gatev                                   LatticeTransferState &State) {
414dd38caf3SYitzhak Mandelbaum   if (auto *HasValueVal = getHasValue(
41548bc7150SMartin Braenne           State.Env, getValueBehindPossiblePointer(
41648bc7150SMartin Braenne                          *CallExpr->getImplicitObjectArgument(), State.Env))) {
417af98b0afSStanislav Gatev     auto &CallExprLoc = State.Env.createStorageLocation(*CallExpr);
418af98b0afSStanislav Gatev     State.Env.setValue(CallExprLoc, *HasValueVal);
419af98b0afSStanislav Gatev     State.Env.setStorageLocation(*CallExpr, CallExprLoc);
420af98b0afSStanislav Gatev   }
421af98b0afSStanislav Gatev }
422af98b0afSStanislav Gatev 
4237f076004SYitzhak Mandelbaum /// `ModelPred` builds a logical formula relating the predicate in
4247f076004SYitzhak Mandelbaum /// `ValueOrPredExpr` to the optional's `has_value` property.
4257f076004SYitzhak Mandelbaum void transferValueOrImpl(const clang::Expr *ValueOrPredExpr,
4267f076004SYitzhak Mandelbaum                          const MatchFinder::MatchResult &Result,
4277f076004SYitzhak Mandelbaum                          LatticeTransferState &State,
4287f076004SYitzhak Mandelbaum                          BoolValue &(*ModelPred)(Environment &Env,
4297f076004SYitzhak Mandelbaum                                                  BoolValue &ExprVal,
4307f076004SYitzhak Mandelbaum                                                  BoolValue &HasValueVal)) {
4317f076004SYitzhak Mandelbaum   auto &Env = State.Env;
4327f076004SYitzhak Mandelbaum 
4337f076004SYitzhak Mandelbaum   const auto *ObjectArgumentExpr =
4347f076004SYitzhak Mandelbaum       Result.Nodes.getNodeAs<clang::CXXMemberCallExpr>(ValueOrCallID)
4357f076004SYitzhak Mandelbaum           ->getImplicitObjectArgument();
4367f076004SYitzhak Mandelbaum 
437dd38caf3SYitzhak Mandelbaum   auto *HasValueVal = getHasValue(
43848bc7150SMartin Braenne       State.Env, getValueBehindPossiblePointer(*ObjectArgumentExpr, State.Env));
439dd38caf3SYitzhak Mandelbaum   if (HasValueVal == nullptr)
4407f076004SYitzhak Mandelbaum     return;
4417f076004SYitzhak Mandelbaum 
442390029beSYitzhak Mandelbaum   Env.addToFlowCondition(
443390029beSYitzhak Mandelbaum       ModelPred(Env, forceBoolValue(Env, *ValueOrPredExpr), *HasValueVal));
4447f076004SYitzhak Mandelbaum }
4457f076004SYitzhak Mandelbaum 
4467f076004SYitzhak Mandelbaum void transferValueOrStringEmptyCall(const clang::Expr *ComparisonExpr,
4477f076004SYitzhak Mandelbaum                                     const MatchFinder::MatchResult &Result,
4487f076004SYitzhak Mandelbaum                                     LatticeTransferState &State) {
4497f076004SYitzhak Mandelbaum   return transferValueOrImpl(ComparisonExpr, Result, State,
4507f076004SYitzhak Mandelbaum                              [](Environment &Env, BoolValue &ExprVal,
4517f076004SYitzhak Mandelbaum                                 BoolValue &HasValueVal) -> BoolValue & {
4527f076004SYitzhak Mandelbaum                                // If the result is *not* empty, then we know the
4537f076004SYitzhak Mandelbaum                                // optional must have been holding a value. If
4547f076004SYitzhak Mandelbaum                                // `ExprVal` is true, though, we don't learn
4557f076004SYitzhak Mandelbaum                                // anything definite about `has_value`, so we
4567f076004SYitzhak Mandelbaum                                // don't add any corresponding implications to
4577f076004SYitzhak Mandelbaum                                // the flow condition.
4587f076004SYitzhak Mandelbaum                                return Env.makeImplication(Env.makeNot(ExprVal),
4597f076004SYitzhak Mandelbaum                                                           HasValueVal);
4607f076004SYitzhak Mandelbaum                              });
4617f076004SYitzhak Mandelbaum }
4627f076004SYitzhak Mandelbaum 
4637f076004SYitzhak Mandelbaum void transferValueOrNotEqX(const Expr *ComparisonExpr,
4647f076004SYitzhak Mandelbaum                            const MatchFinder::MatchResult &Result,
4657f076004SYitzhak Mandelbaum                            LatticeTransferState &State) {
4667f076004SYitzhak Mandelbaum   transferValueOrImpl(ComparisonExpr, Result, State,
4677f076004SYitzhak Mandelbaum                       [](Environment &Env, BoolValue &ExprVal,
4687f076004SYitzhak Mandelbaum                          BoolValue &HasValueVal) -> BoolValue & {
4697f076004SYitzhak Mandelbaum                         // We know that if `(opt.value_or(X) != X)` then
4707f076004SYitzhak Mandelbaum                         // `opt.hasValue()`, even without knowing further
4717f076004SYitzhak Mandelbaum                         // details about the contents of `opt`.
4727f076004SYitzhak Mandelbaum                         return Env.makeImplication(ExprVal, HasValueVal);
4737f076004SYitzhak Mandelbaum                       });
4747f076004SYitzhak Mandelbaum }
4757f076004SYitzhak Mandelbaum 
47665e710c3SStanislav Gatev void transferCallReturningOptional(const CallExpr *E,
47765e710c3SStanislav Gatev                                    const MatchFinder::MatchResult &Result,
47865e710c3SStanislav Gatev                                    LatticeTransferState &State) {
47965e710c3SStanislav Gatev   if (State.Env.getStorageLocation(*E, SkipPast::None) != nullptr)
48065e710c3SStanislav Gatev     return;
48165e710c3SStanislav Gatev 
48265e710c3SStanislav Gatev   auto &Loc = State.Env.createStorageLocation(*E);
48365e710c3SStanislav Gatev   State.Env.setStorageLocation(*E, Loc);
48465e710c3SStanislav Gatev   State.Env.setValue(
48565e710c3SStanislav Gatev       Loc, createOptionalValue(State.Env, State.Env.makeAtomicBoolValue()));
48665e710c3SStanislav Gatev }
48765e710c3SStanislav Gatev 
4880e8d4a6dSYitzhak Mandelbaum void assignOptionalValue(const Expr &E, Environment &Env,
489092a530cSStanislav Gatev                          BoolValue &HasValueVal) {
49048bc7150SMartin Braenne   if (auto *OptionalLoc = maybeSkipPointer(
49148bc7150SMartin Braenne           Env.getStorageLocation(E, SkipPast::Reference), Env)) {
4920e8d4a6dSYitzhak Mandelbaum     Env.setValue(*OptionalLoc, createOptionalValue(Env, HasValueVal));
4939e0fc676SStanislav Gatev   }
4949e0fc676SStanislav Gatev }
4959e0fc676SStanislav Gatev 
496b000b770SStanislav Gatev /// Returns a symbolic value for the "has_value" property of an `optional<T>`
497b000b770SStanislav Gatev /// value that is constructed/assigned from a value of type `U` or `optional<U>`
498b000b770SStanislav Gatev /// where `T` is constructible from `U`.
499390029beSYitzhak Mandelbaum BoolValue &valueOrConversionHasValue(const FunctionDecl &F, const Expr &E,
500b000b770SStanislav Gatev                                      const MatchFinder::MatchResult &MatchRes,
501b000b770SStanislav Gatev                                      LatticeTransferState &State) {
5020086a355SYitzhak Mandelbaum   assert(F.getTemplateSpecializationArgs() != nullptr);
503b000b770SStanislav Gatev   assert(F.getTemplateSpecializationArgs()->size() > 0);
504b000b770SStanislav Gatev 
505c849843cSMartin Braenne   const int TemplateParamOptionalWrappersCount =
506c849843cSMartin Braenne       countOptionalWrappers(*MatchRes.Context, F.getTemplateSpecializationArgs()
507c849843cSMartin Braenne                                                    ->get(0)
508c849843cSMartin Braenne                                                    .getAsType()
509c849843cSMartin Braenne                                                    .getNonReferenceType());
510c849843cSMartin Braenne   const int ArgTypeOptionalWrappersCount = countOptionalWrappers(
511c849843cSMartin Braenne       *MatchRes.Context, E.getType().getNonReferenceType());
512b000b770SStanislav Gatev 
513b000b770SStanislav Gatev   // Check if this is a constructor/assignment call for `optional<T>` with
514b000b770SStanislav Gatev   // argument of type `U` such that `T` is constructible from `U`.
515b000b770SStanislav Gatev   if (TemplateParamOptionalWrappersCount == ArgTypeOptionalWrappersCount)
516b000b770SStanislav Gatev     return State.Env.getBoolLiteralValue(true);
517b000b770SStanislav Gatev 
518b000b770SStanislav Gatev   // This is a constructor/assignment call for `optional<T>` with argument of
519b000b770SStanislav Gatev   // type `optional<U>` such that `T` is constructible from `U`.
520dd38caf3SYitzhak Mandelbaum   if (auto *HasValueVal =
521dd38caf3SYitzhak Mandelbaum           getHasValue(State.Env, State.Env.getValue(E, SkipPast::Reference)))
522dd38caf3SYitzhak Mandelbaum     return *HasValueVal;
523b000b770SStanislav Gatev   return State.Env.makeAtomicBoolValue();
524b000b770SStanislav Gatev }
525b000b770SStanislav Gatev 
526092a530cSStanislav Gatev void transferValueOrConversionConstructor(
527092a530cSStanislav Gatev     const CXXConstructExpr *E, const MatchFinder::MatchResult &MatchRes,
5289e0fc676SStanislav Gatev     LatticeTransferState &State) {
529092a530cSStanislav Gatev   assert(E->getNumArgs() > 0);
530092a530cSStanislav Gatev 
5310e8d4a6dSYitzhak Mandelbaum   assignOptionalValue(*E, State.Env,
532390029beSYitzhak Mandelbaum                       valueOrConversionHasValue(*E->getConstructor(),
533b000b770SStanislav Gatev                                                 *E->getArg(0), MatchRes,
534b000b770SStanislav Gatev                                                 State));
535b000b770SStanislav Gatev }
536092a530cSStanislav Gatev 
537b000b770SStanislav Gatev void transferAssignment(const CXXOperatorCallExpr *E, BoolValue &HasValueVal,
538b000b770SStanislav Gatev                         LatticeTransferState &State) {
539b000b770SStanislav Gatev   assert(E->getNumArgs() > 0);
540b000b770SStanislav Gatev 
541b000b770SStanislav Gatev   auto *OptionalLoc =
542b000b770SStanislav Gatev       State.Env.getStorageLocation(*E->getArg(0), SkipPast::Reference);
543a9ad689eSSam Estep   if (OptionalLoc == nullptr)
544a9ad689eSSam Estep     return;
545b000b770SStanislav Gatev 
546b000b770SStanislav Gatev   State.Env.setValue(*OptionalLoc, createOptionalValue(State.Env, HasValueVal));
547b000b770SStanislav Gatev 
548b000b770SStanislav Gatev   // Assign a storage location for the whole expression.
549b000b770SStanislav Gatev   State.Env.setStorageLocation(*E, *OptionalLoc);
550b000b770SStanislav Gatev }
551b000b770SStanislav Gatev 
552b000b770SStanislav Gatev void transferValueOrConversionAssignment(
553b000b770SStanislav Gatev     const CXXOperatorCallExpr *E, const MatchFinder::MatchResult &MatchRes,
554b000b770SStanislav Gatev     LatticeTransferState &State) {
555b000b770SStanislav Gatev   assert(E->getNumArgs() > 1);
556b000b770SStanislav Gatev   transferAssignment(E,
557390029beSYitzhak Mandelbaum                      valueOrConversionHasValue(*E->getDirectCallee(),
55806decd0bSKazu Hirata                                                *E->getArg(1), MatchRes, State),
559b000b770SStanislav Gatev                      State);
560b000b770SStanislav Gatev }
561b000b770SStanislav Gatev 
562b000b770SStanislav Gatev void transferNulloptAssignment(const CXXOperatorCallExpr *E,
563b000b770SStanislav Gatev                                const MatchFinder::MatchResult &,
564b000b770SStanislav Gatev                                LatticeTransferState &State) {
565b000b770SStanislav Gatev   transferAssignment(E, State.Env.getBoolLiteralValue(false), State);
5669e0fc676SStanislav Gatev }
5679e0fc676SStanislav Gatev 
56848bc7150SMartin Braenne void transferSwap(StorageLocation *Loc1, StorageLocation *Loc2,
569d4fb829bSYitzhak Mandelbaum                   Environment &Env) {
570d4fb829bSYitzhak Mandelbaum   // We account for cases where one or both of the optionals are not modeled,
571d4fb829bSYitzhak Mandelbaum   // either lacking associated storage locations, or lacking values associated
572d4fb829bSYitzhak Mandelbaum   // to such storage locations.
5732ddd57aeSStanislav Gatev 
574d4fb829bSYitzhak Mandelbaum   if (Loc1 == nullptr) {
575d4fb829bSYitzhak Mandelbaum     if (Loc2 != nullptr)
576d4fb829bSYitzhak Mandelbaum       Env.setValue(*Loc2, createOptionalValue(Env, Env.makeAtomicBoolValue()));
577d4fb829bSYitzhak Mandelbaum     return;
578d4fb829bSYitzhak Mandelbaum   }
579d4fb829bSYitzhak Mandelbaum   if (Loc2 == nullptr) {
580d4fb829bSYitzhak Mandelbaum     Env.setValue(*Loc1, createOptionalValue(Env, Env.makeAtomicBoolValue()));
581d4fb829bSYitzhak Mandelbaum     return;
582d4fb829bSYitzhak Mandelbaum   }
5832ddd57aeSStanislav Gatev 
584d4fb829bSYitzhak Mandelbaum   // Both expressions have locations, though they may not have corresponding
585d4fb829bSYitzhak Mandelbaum   // values. In that case, we create a fresh value at this point. Note that if
586d4fb829bSYitzhak Mandelbaum   // two branches both do this, they will not share the value, but it at least
587d4fb829bSYitzhak Mandelbaum   // allows for local reasoning about the value. To avoid the above, we would
588d4fb829bSYitzhak Mandelbaum   // need *lazy* value allocation.
589d4fb829bSYitzhak Mandelbaum   // FIXME: allocate values lazily, instead of just creating a fresh value.
590d4fb829bSYitzhak Mandelbaum   auto *Val1 = Env.getValue(*Loc1);
591d4fb829bSYitzhak Mandelbaum   if (Val1 == nullptr)
592d4fb829bSYitzhak Mandelbaum     Val1 = &createOptionalValue(Env, Env.makeAtomicBoolValue());
593d4fb829bSYitzhak Mandelbaum 
594d4fb829bSYitzhak Mandelbaum   auto *Val2 = Env.getValue(*Loc2);
595d4fb829bSYitzhak Mandelbaum   if (Val2 == nullptr)
596d4fb829bSYitzhak Mandelbaum     Val2 = &createOptionalValue(Env, Env.makeAtomicBoolValue());
597d4fb829bSYitzhak Mandelbaum 
598d4fb829bSYitzhak Mandelbaum   Env.setValue(*Loc1, *Val2);
599d4fb829bSYitzhak Mandelbaum   Env.setValue(*Loc2, *Val1);
6002ddd57aeSStanislav Gatev }
6012ddd57aeSStanislav Gatev 
6022ddd57aeSStanislav Gatev void transferSwapCall(const CXXMemberCallExpr *E,
6032ddd57aeSStanislav Gatev                       const MatchFinder::MatchResult &,
6042ddd57aeSStanislav Gatev                       LatticeTransferState &State) {
6052ddd57aeSStanislav Gatev   assert(E->getNumArgs() == 1);
60648bc7150SMartin Braenne   transferSwap(maybeSkipPointer(
60748bc7150SMartin Braenne                    State.Env.getStorageLocation(*E->getImplicitObjectArgument(),
60848bc7150SMartin Braenne                                                 SkipPast::Reference),
60948bc7150SMartin Braenne                    State.Env),
61048bc7150SMartin Braenne                State.Env.getStorageLocation(*E->getArg(0), SkipPast::Reference),
61148bc7150SMartin Braenne                State.Env);
6122ddd57aeSStanislav Gatev }
6132ddd57aeSStanislav Gatev 
6142ddd57aeSStanislav Gatev void transferStdSwapCall(const CallExpr *E, const MatchFinder::MatchResult &,
6152ddd57aeSStanislav Gatev                          LatticeTransferState &State) {
6162ddd57aeSStanislav Gatev   assert(E->getNumArgs() == 2);
61748bc7150SMartin Braenne   transferSwap(State.Env.getStorageLocation(*E->getArg(0), SkipPast::Reference),
61848bc7150SMartin Braenne                State.Env.getStorageLocation(*E->getArg(1), SkipPast::Reference),
61948bc7150SMartin Braenne                State.Env);
6202ddd57aeSStanislav Gatev }
6212ddd57aeSStanislav Gatev 
62225956d55SAMS21 void transferStdForwardCall(const CallExpr *E, const MatchFinder::MatchResult &,
62325956d55SAMS21                             LatticeTransferState &State) {
62425956d55SAMS21   assert(E->getNumArgs() == 1);
62525956d55SAMS21 
62625956d55SAMS21   StorageLocation *LocRet = State.Env.getStorageLocation(*E, SkipPast::None);
62725956d55SAMS21   if (LocRet != nullptr)
62825956d55SAMS21     return;
62925956d55SAMS21 
63025956d55SAMS21   StorageLocation *LocArg =
63125956d55SAMS21       State.Env.getStorageLocation(*E->getArg(0), SkipPast::Reference);
63225956d55SAMS21 
63325956d55SAMS21   if (LocArg == nullptr)
63425956d55SAMS21     return;
63525956d55SAMS21 
63625956d55SAMS21   Value *ValArg = State.Env.getValue(*LocArg);
63725956d55SAMS21   if (ValArg == nullptr)
63825956d55SAMS21     ValArg = &createOptionalValue(State.Env, State.Env.makeAtomicBoolValue());
63925956d55SAMS21 
64025956d55SAMS21   // Create a new storage location
64125956d55SAMS21   LocRet = &State.Env.createStorageLocation(*E);
64225956d55SAMS21   State.Env.setStorageLocation(*E, *LocRet);
64325956d55SAMS21 
64425956d55SAMS21   State.Env.setValue(*LocRet, *ValArg);
64525956d55SAMS21 }
64625956d55SAMS21 
647390029beSYitzhak Mandelbaum BoolValue &evaluateEquality(Environment &Env, BoolValue &EqVal, BoolValue &LHS,
648390029beSYitzhak Mandelbaum                             BoolValue &RHS) {
649390029beSYitzhak Mandelbaum   // Logically, an optional<T> object is composed of two values - a `has_value`
650390029beSYitzhak Mandelbaum   // bit and a value of type T. Equality of optional objects compares both
651390029beSYitzhak Mandelbaum   // values. Therefore, merely comparing the `has_value` bits isn't sufficient:
652390029beSYitzhak Mandelbaum   // when two optional objects are engaged, the equality of their respective
653390029beSYitzhak Mandelbaum   // values of type T matters. Since we only track the `has_value` bits, we
654390029beSYitzhak Mandelbaum   // can't make any conclusions about equality when we know that two optional
655390029beSYitzhak Mandelbaum   // objects are engaged.
656390029beSYitzhak Mandelbaum   //
657390029beSYitzhak Mandelbaum   // We express this as two facts about the equality:
658390029beSYitzhak Mandelbaum   // a) EqVal => (LHS & RHS) v (!RHS & !LHS)
659390029beSYitzhak Mandelbaum   //    If they are equal, then either both are set or both are unset.
660390029beSYitzhak Mandelbaum   // b) (!LHS & !RHS) => EqVal
661390029beSYitzhak Mandelbaum   //    If neither is set, then they are equal.
662390029beSYitzhak Mandelbaum   // We rewrite b) as !EqVal => (LHS v RHS), for a more compact formula.
663390029beSYitzhak Mandelbaum   return Env.makeAnd(
664390029beSYitzhak Mandelbaum       Env.makeImplication(
665390029beSYitzhak Mandelbaum           EqVal, Env.makeOr(Env.makeAnd(LHS, RHS),
666390029beSYitzhak Mandelbaum                             Env.makeAnd(Env.makeNot(LHS), Env.makeNot(RHS)))),
667390029beSYitzhak Mandelbaum       Env.makeImplication(Env.makeNot(EqVal), Env.makeOr(LHS, RHS)));
668390029beSYitzhak Mandelbaum }
669390029beSYitzhak Mandelbaum 
670390029beSYitzhak Mandelbaum void transferOptionalAndOptionalCmp(const clang::CXXOperatorCallExpr *CmpExpr,
671390029beSYitzhak Mandelbaum                                     const MatchFinder::MatchResult &,
672390029beSYitzhak Mandelbaum                                     LatticeTransferState &State) {
673390029beSYitzhak Mandelbaum   Environment &Env = State.Env;
674390029beSYitzhak Mandelbaum   auto *CmpValue = &forceBoolValue(Env, *CmpExpr);
675390029beSYitzhak Mandelbaum   if (auto *LHasVal = getHasValue(
676390029beSYitzhak Mandelbaum           Env, Env.getValue(*CmpExpr->getArg(0), SkipPast::Reference)))
677390029beSYitzhak Mandelbaum     if (auto *RHasVal = getHasValue(
678390029beSYitzhak Mandelbaum             Env, Env.getValue(*CmpExpr->getArg(1), SkipPast::Reference))) {
679390029beSYitzhak Mandelbaum       if (CmpExpr->getOperator() == clang::OO_ExclaimEqual)
680390029beSYitzhak Mandelbaum         CmpValue = &State.Env.makeNot(*CmpValue);
681390029beSYitzhak Mandelbaum       Env.addToFlowCondition(
682390029beSYitzhak Mandelbaum           evaluateEquality(Env, *CmpValue, *LHasVal, *RHasVal));
683390029beSYitzhak Mandelbaum     }
684390029beSYitzhak Mandelbaum }
685390029beSYitzhak Mandelbaum 
686390029beSYitzhak Mandelbaum void transferOptionalAndValueCmp(const clang::CXXOperatorCallExpr *CmpExpr,
687390029beSYitzhak Mandelbaum                                  const clang::Expr *E, Environment &Env) {
688390029beSYitzhak Mandelbaum   auto *CmpValue = &forceBoolValue(Env, *CmpExpr);
689390029beSYitzhak Mandelbaum   if (auto *HasVal = getHasValue(Env, Env.getValue(*E, SkipPast::Reference))) {
690390029beSYitzhak Mandelbaum     if (CmpExpr->getOperator() == clang::OO_ExclaimEqual)
691390029beSYitzhak Mandelbaum       CmpValue = &Env.makeNot(*CmpValue);
692390029beSYitzhak Mandelbaum     Env.addToFlowCondition(evaluateEquality(Env, *CmpValue, *HasVal,
693390029beSYitzhak Mandelbaum                                             Env.getBoolLiteralValue(true)));
694390029beSYitzhak Mandelbaum   }
695390029beSYitzhak Mandelbaum }
696390029beSYitzhak Mandelbaum 
6976ad0788cSKazu Hirata std::optional<StatementMatcher>
698a184a0d8SYitzhak Mandelbaum ignorableOptional(const UncheckedOptionalAccessModelOptions &Options) {
6995d22d1f5SYitzhak Mandelbaum   if (Options.IgnoreSmartPointerDereference) {
7005d22d1f5SYitzhak Mandelbaum     auto SmartPtrUse = expr(ignoringParenImpCasts(cxxOperatorCallExpr(
7015d22d1f5SYitzhak Mandelbaum         anyOf(hasOverloadedOperatorName("->"), hasOverloadedOperatorName("*")),
7025d22d1f5SYitzhak Mandelbaum         unless(hasArgument(0, expr(hasOptionalType()))))));
7035d22d1f5SYitzhak Mandelbaum     return expr(
7045d22d1f5SYitzhak Mandelbaum         anyOf(SmartPtrUse, memberExpr(hasObjectExpression(SmartPtrUse))));
7055d22d1f5SYitzhak Mandelbaum   }
70634e0d057SKazu Hirata   return std::nullopt;
707a184a0d8SYitzhak Mandelbaum }
708a184a0d8SYitzhak Mandelbaum 
70958fe7f96SSam Estep StatementMatcher
7106ad0788cSKazu Hirata valueCall(const std::optional<StatementMatcher> &IgnorableOptional) {
71158fe7f96SSam Estep   return isOptionalMemberCallWithName("value", IgnorableOptional);
71258fe7f96SSam Estep }
71358fe7f96SSam Estep 
71458fe7f96SSam Estep StatementMatcher
7156ad0788cSKazu Hirata valueOperatorCall(const std::optional<StatementMatcher> &IgnorableOptional) {
71658fe7f96SSam Estep   return expr(anyOf(isOptionalOperatorCallWithName("*", IgnorableOptional),
71758fe7f96SSam Estep                     isOptionalOperatorCallWithName("->", IgnorableOptional)));
71858fe7f96SSam Estep }
71958fe7f96SSam Estep 
7205d22d1f5SYitzhak Mandelbaum auto buildTransferMatchSwitch() {
721b000b770SStanislav Gatev   // FIXME: Evaluate the efficiency of matchers. If using matchers results in a
722b000b770SStanislav Gatev   // lot of duplicated work (e.g. string comparisons), consider providing APIs
723b000b770SStanislav Gatev   // that avoid it through memoization.
7247538b360SWei Yi Tee   return CFGMatchSwitchBuilder<LatticeTransferState>()
725af98b0afSStanislav Gatev       // Attach a symbolic "has_value" state to optional values that we see for
726af98b0afSStanislav Gatev       // the first time.
7277538b360SWei Yi Tee       .CaseOfCFGStmt<Expr>(
7286adfc64eSYitzhak Mandelbaum           expr(anyOf(declRefExpr(), memberExpr()), hasOptionalType()),
729af98b0afSStanislav Gatev           initializeOptionalReference)
730af98b0afSStanislav Gatev 
7319e0fc676SStanislav Gatev       // make_optional
7327538b360SWei Yi Tee       .CaseOfCFGStmt<CallExpr>(isMakeOptionalCall(), transferMakeOptionalCall)
733092a530cSStanislav Gatev 
7340e8d4a6dSYitzhak Mandelbaum       // optional::optional (in place)
7357538b360SWei Yi Tee       .CaseOfCFGStmt<CXXConstructExpr>(
736092a530cSStanislav Gatev           isOptionalInPlaceConstructor(),
737092a530cSStanislav Gatev           [](const CXXConstructExpr *E, const MatchFinder::MatchResult &,
738092a530cSStanislav Gatev              LatticeTransferState &State) {
7390e8d4a6dSYitzhak Mandelbaum             assignOptionalValue(*E, State.Env,
7400e8d4a6dSYitzhak Mandelbaum                                 State.Env.getBoolLiteralValue(true));
741092a530cSStanislav Gatev           })
7420e8d4a6dSYitzhak Mandelbaum       // nullopt_t::nullopt_t
7437538b360SWei Yi Tee       .CaseOfCFGStmt<CXXConstructExpr>(
744390029beSYitzhak Mandelbaum           isNulloptConstructor(),
745092a530cSStanislav Gatev           [](const CXXConstructExpr *E, const MatchFinder::MatchResult &,
746092a530cSStanislav Gatev              LatticeTransferState &State) {
7470e8d4a6dSYitzhak Mandelbaum             assignOptionalValue(*E, State.Env,
748092a530cSStanislav Gatev                                 State.Env.getBoolLiteralValue(false));
749092a530cSStanislav Gatev           })
7500e8d4a6dSYitzhak Mandelbaum       // optional::optional(nullopt_t)
751390029beSYitzhak Mandelbaum       .CaseOfCFGStmt<CXXConstructExpr>(
752390029beSYitzhak Mandelbaum           isOptionalNulloptConstructor(),
753390029beSYitzhak Mandelbaum           [](const CXXConstructExpr *E, const MatchFinder::MatchResult &,
754390029beSYitzhak Mandelbaum              LatticeTransferState &State) {
7550e8d4a6dSYitzhak Mandelbaum             assignOptionalValue(*E, State.Env,
7560e8d4a6dSYitzhak Mandelbaum                                 State.Env.getBoolLiteralValue(false));
757390029beSYitzhak Mandelbaum           })
7580e8d4a6dSYitzhak Mandelbaum       // optional::optional (value/conversion)
7597538b360SWei Yi Tee       .CaseOfCFGStmt<CXXConstructExpr>(isOptionalValueOrConversionConstructor(),
760092a530cSStanislav Gatev                                        transferValueOrConversionConstructor)
7619e0fc676SStanislav Gatev 
762b000b770SStanislav Gatev       // optional::operator=
7637538b360SWei Yi Tee       .CaseOfCFGStmt<CXXOperatorCallExpr>(
7647538b360SWei Yi Tee           isOptionalValueOrConversionAssignment(),
765b000b770SStanislav Gatev           transferValueOrConversionAssignment)
7667538b360SWei Yi Tee       .CaseOfCFGStmt<CXXOperatorCallExpr>(isOptionalNulloptAssignment(),
767b000b770SStanislav Gatev                                           transferNulloptAssignment)
768b000b770SStanislav Gatev 
769af98b0afSStanislav Gatev       // optional::value
7707538b360SWei Yi Tee       .CaseOfCFGStmt<CXXMemberCallExpr>(
7715d22d1f5SYitzhak Mandelbaum           valueCall(std::nullopt),
772092a530cSStanislav Gatev           [](const CXXMemberCallExpr *E, const MatchFinder::MatchResult &,
773092a530cSStanislav Gatev              LatticeTransferState &State) {
774af98b0afSStanislav Gatev             transferUnwrapCall(E, E->getImplicitObjectArgument(), State);
775af98b0afSStanislav Gatev           })
776af98b0afSStanislav Gatev 
777af98b0afSStanislav Gatev       // optional::operator*, optional::operator->
77848bc7150SMartin Braenne       // FIXME: This does something slightly strange for `operator->`.
77948bc7150SMartin Braenne       // `transferUnwrapCall()` may create a new value of type `T` for the
78048bc7150SMartin Braenne       // `optional<T>`, and it associates that value with `E`. In the case of
78148bc7150SMartin Braenne       // `operator->`, `E` is a pointer. As a result, we associate an
78248bc7150SMartin Braenne       // expression of pointer type with a storage location of non-pointer type
78348bc7150SMartin Braenne       // `T`. This can confound other code that expects expressions of
78448bc7150SMartin Braenne       // pointer type to be associated with `PointerValue`s, such as the
78548bc7150SMartin Braenne       // centrally provided accessors `getImplicitObjectLocation()` and
78648bc7150SMartin Braenne       // `getBaseObjectLocation()`, and this is the reason we need to use our
78748bc7150SMartin Braenne       // own 'maybeSkipPointer()` and `getValueBehindPossiblePointer()` instead
78848bc7150SMartin Braenne       // of these accessors.
7895d22d1f5SYitzhak Mandelbaum       .CaseOfCFGStmt<CallExpr>(valueOperatorCall(std::nullopt),
7907538b360SWei Yi Tee                                [](const CallExpr *E,
7917538b360SWei Yi Tee                                   const MatchFinder::MatchResult &,
792092a530cSStanislav Gatev                                   LatticeTransferState &State) {
793af98b0afSStanislav Gatev                                  transferUnwrapCall(E, E->getArg(0), State);
794af98b0afSStanislav Gatev                                })
795af98b0afSStanislav Gatev 
796af98b0afSStanislav Gatev       // optional::has_value
7977538b360SWei Yi Tee       .CaseOfCFGStmt<CXXMemberCallExpr>(
7987538b360SWei Yi Tee           isOptionalMemberCallWithName("has_value"),
799af98b0afSStanislav Gatev           transferOptionalHasValueCall)
800af98b0afSStanislav Gatev 
8019e0fc676SStanislav Gatev       // optional::operator bool
8027538b360SWei Yi Tee       .CaseOfCFGStmt<CXXMemberCallExpr>(
8037538b360SWei Yi Tee           isOptionalMemberCallWithName("operator bool"),
8049e0fc676SStanislav Gatev           transferOptionalHasValueCall)
8059e0fc676SStanislav Gatev 
8069e0fc676SStanislav Gatev       // optional::emplace
8077538b360SWei Yi Tee       .CaseOfCFGStmt<CXXMemberCallExpr>(
808092a530cSStanislav Gatev           isOptionalMemberCallWithName("emplace"),
809092a530cSStanislav Gatev           [](const CXXMemberCallExpr *E, const MatchFinder::MatchResult &,
810092a530cSStanislav Gatev              LatticeTransferState &State) {
8110e8d4a6dSYitzhak Mandelbaum             assignOptionalValue(*E->getImplicitObjectArgument(), State.Env,
812092a530cSStanislav Gatev                                 State.Env.getBoolLiteralValue(true));
813092a530cSStanislav Gatev           })
8149e0fc676SStanislav Gatev 
8159e0fc676SStanislav Gatev       // optional::reset
8167538b360SWei Yi Tee       .CaseOfCFGStmt<CXXMemberCallExpr>(
817092a530cSStanislav Gatev           isOptionalMemberCallWithName("reset"),
818092a530cSStanislav Gatev           [](const CXXMemberCallExpr *E, const MatchFinder::MatchResult &,
819092a530cSStanislav Gatev              LatticeTransferState &State) {
8200e8d4a6dSYitzhak Mandelbaum             assignOptionalValue(*E->getImplicitObjectArgument(), State.Env,
821092a530cSStanislav Gatev                                 State.Env.getBoolLiteralValue(false));
822092a530cSStanislav Gatev           })
8239e0fc676SStanislav Gatev 
8242ddd57aeSStanislav Gatev       // optional::swap
8257538b360SWei Yi Tee       .CaseOfCFGStmt<CXXMemberCallExpr>(isOptionalMemberCallWithName("swap"),
8262ddd57aeSStanislav Gatev                                         transferSwapCall)
8272ddd57aeSStanislav Gatev 
8282ddd57aeSStanislav Gatev       // std::swap
8297538b360SWei Yi Tee       .CaseOfCFGStmt<CallExpr>(isStdSwapCall(), transferStdSwapCall)
8302ddd57aeSStanislav Gatev 
83125956d55SAMS21       // std::forward
83225956d55SAMS21       .CaseOfCFGStmt<CallExpr>(isStdForwardCall(), transferStdForwardCall)
83325956d55SAMS21 
8347f076004SYitzhak Mandelbaum       // opt.value_or("").empty()
8357538b360SWei Yi Tee       .CaseOfCFGStmt<Expr>(isValueOrStringEmptyCall(),
8367538b360SWei Yi Tee                            transferValueOrStringEmptyCall)
8377f076004SYitzhak Mandelbaum 
8387f076004SYitzhak Mandelbaum       // opt.value_or(X) != X
8397538b360SWei Yi Tee       .CaseOfCFGStmt<Expr>(isValueOrNotEqX(), transferValueOrNotEqX)
8407f076004SYitzhak Mandelbaum 
841390029beSYitzhak Mandelbaum       // Comparisons (==, !=):
842390029beSYitzhak Mandelbaum       .CaseOfCFGStmt<CXXOperatorCallExpr>(
843390029beSYitzhak Mandelbaum           isComparisonOperatorCall(hasAnyOptionalType(), hasAnyOptionalType()),
844390029beSYitzhak Mandelbaum           transferOptionalAndOptionalCmp)
845390029beSYitzhak Mandelbaum       .CaseOfCFGStmt<CXXOperatorCallExpr>(
846390029beSYitzhak Mandelbaum           isComparisonOperatorCall(hasOptionalType(),
847390029beSYitzhak Mandelbaum                                    unless(hasAnyOptionalType())),
848390029beSYitzhak Mandelbaum           [](const clang::CXXOperatorCallExpr *Cmp,
849390029beSYitzhak Mandelbaum              const MatchFinder::MatchResult &, LatticeTransferState &State) {
850390029beSYitzhak Mandelbaum             transferOptionalAndValueCmp(Cmp, Cmp->getArg(0), State.Env);
851390029beSYitzhak Mandelbaum           })
852390029beSYitzhak Mandelbaum       .CaseOfCFGStmt<CXXOperatorCallExpr>(
853390029beSYitzhak Mandelbaum           isComparisonOperatorCall(unless(hasAnyOptionalType()),
854390029beSYitzhak Mandelbaum                                    hasOptionalType()),
855390029beSYitzhak Mandelbaum           [](const clang::CXXOperatorCallExpr *Cmp,
856390029beSYitzhak Mandelbaum              const MatchFinder::MatchResult &, LatticeTransferState &State) {
857390029beSYitzhak Mandelbaum             transferOptionalAndValueCmp(Cmp, Cmp->getArg(1), State.Env);
858390029beSYitzhak Mandelbaum           })
859390029beSYitzhak Mandelbaum 
86065e710c3SStanislav Gatev       // returns optional
8617538b360SWei Yi Tee       .CaseOfCFGStmt<CallExpr>(isCallReturningOptional(),
86265e710c3SStanislav Gatev                                transferCallReturningOptional)
86365e710c3SStanislav Gatev 
864af98b0afSStanislav Gatev       .Build();
865af98b0afSStanislav Gatev }
866af98b0afSStanislav Gatev 
86758fe7f96SSam Estep std::vector<SourceLocation> diagnoseUnwrapCall(const Expr *UnwrapExpr,
86858fe7f96SSam Estep                                                const Expr *ObjectExpr,
86958fe7f96SSam Estep                                                const Environment &Env) {
87048bc7150SMartin Braenne   if (auto *OptionalVal = getValueBehindPossiblePointer(*ObjectExpr, Env)) {
87158fe7f96SSam Estep     auto *Prop = OptionalVal->getProperty("has_value");
87258fe7f96SSam Estep     if (auto *HasValueVal = cast_or_null<BoolValue>(Prop)) {
87358fe7f96SSam Estep       if (Env.flowConditionImplies(*HasValueVal))
87458fe7f96SSam Estep         return {};
87558fe7f96SSam Estep     }
87658fe7f96SSam Estep   }
87758fe7f96SSam Estep 
87858fe7f96SSam Estep   // Record that this unwrap is *not* provably safe.
87958fe7f96SSam Estep   // FIXME: include either the name of the optional (if applicable) or a source
88058fe7f96SSam Estep   // range of the access for easier interpretation of the result.
88158fe7f96SSam Estep   return {ObjectExpr->getBeginLoc()};
88258fe7f96SSam Estep }
88358fe7f96SSam Estep 
88458fe7f96SSam Estep auto buildDiagnoseMatchSwitch(
88558fe7f96SSam Estep     const UncheckedOptionalAccessModelOptions &Options) {
88658fe7f96SSam Estep   // FIXME: Evaluate the efficiency of matchers. If using matchers results in a
88758fe7f96SSam Estep   // lot of duplicated work (e.g. string comparisons), consider providing APIs
88858fe7f96SSam Estep   // that avoid it through memoization.
88958fe7f96SSam Estep   auto IgnorableOptional = ignorableOptional(Options);
8907538b360SWei Yi Tee   return CFGMatchSwitchBuilder<const Environment, std::vector<SourceLocation>>()
89158fe7f96SSam Estep       // optional::value
8927538b360SWei Yi Tee       .CaseOfCFGStmt<CXXMemberCallExpr>(
89358fe7f96SSam Estep           valueCall(IgnorableOptional),
89458fe7f96SSam Estep           [](const CXXMemberCallExpr *E, const MatchFinder::MatchResult &,
89558fe7f96SSam Estep              const Environment &Env) {
89658fe7f96SSam Estep             return diagnoseUnwrapCall(E, E->getImplicitObjectArgument(), Env);
89758fe7f96SSam Estep           })
89858fe7f96SSam Estep 
89958fe7f96SSam Estep       // optional::operator*, optional::operator->
9007538b360SWei Yi Tee       .CaseOfCFGStmt<CallExpr>(
90158fe7f96SSam Estep           valueOperatorCall(IgnorableOptional),
90258fe7f96SSam Estep           [](const CallExpr *E, const MatchFinder::MatchResult &,
90358fe7f96SSam Estep              const Environment &Env) {
90458fe7f96SSam Estep             return diagnoseUnwrapCall(E, E->getArg(0), Env);
90558fe7f96SSam Estep           })
90658fe7f96SSam Estep       .Build();
90758fe7f96SSam Estep }
90858fe7f96SSam Estep 
909af98b0afSStanislav Gatev } // namespace
910af98b0afSStanislav Gatev 
9117e63a0d4SYitzhak Mandelbaum ast_matchers::DeclarationMatcher
9127e63a0d4SYitzhak Mandelbaum UncheckedOptionalAccessModel::optionalClassDecl() {
9137e63a0d4SYitzhak Mandelbaum   return optionalClass();
9147e63a0d4SYitzhak Mandelbaum }
9157e63a0d4SYitzhak Mandelbaum 
9165d22d1f5SYitzhak Mandelbaum UncheckedOptionalAccessModel::UncheckedOptionalAccessModel(ASTContext &Ctx)
917cf1f978dSSam Estep     : DataflowAnalysis<UncheckedOptionalAccessModel, NoopLattice>(Ctx),
9185d22d1f5SYitzhak Mandelbaum       TransferMatchSwitch(buildTransferMatchSwitch()) {}
919af98b0afSStanislav Gatev 
9206b991ba4SYitzhak Mandelbaum void UncheckedOptionalAccessModel::transfer(const CFGElement &Elt,
9217538b360SWei Yi Tee                                             NoopLattice &L, Environment &Env) {
922af98b0afSStanislav Gatev   LatticeTransferState State(L, Env);
9236b991ba4SYitzhak Mandelbaum   TransferMatchSwitch(Elt, getASTContext(), State);
924af98b0afSStanislav Gatev }
925af98b0afSStanislav Gatev 
926c0725865SYitzhak Mandelbaum ComparisonResult UncheckedOptionalAccessModel::compare(
927c0725865SYitzhak Mandelbaum     QualType Type, const Value &Val1, const Environment &Env1,
928c0725865SYitzhak Mandelbaum     const Value &Val2, const Environment &Env2) {
929c0725865SYitzhak Mandelbaum   if (!isOptionalType(Type))
930c0725865SYitzhak Mandelbaum     return ComparisonResult::Unknown;
931d34fbf2dSYitzhak Mandelbaum   bool MustNonEmpty1 = isNonEmptyOptional(Val1, Env1);
932d34fbf2dSYitzhak Mandelbaum   bool MustNonEmpty2 = isNonEmptyOptional(Val2, Env2);
93325956d55SAMS21   if (MustNonEmpty1 && MustNonEmpty2)
93425956d55SAMS21     return ComparisonResult::Same;
935d34fbf2dSYitzhak Mandelbaum   // If exactly one is true, then they're different, no reason to check whether
936d34fbf2dSYitzhak Mandelbaum   // they're definitely empty.
93725956d55SAMS21   if (MustNonEmpty1 || MustNonEmpty2)
93825956d55SAMS21     return ComparisonResult::Different;
939d34fbf2dSYitzhak Mandelbaum   // Check if they're both definitely empty.
940d34fbf2dSYitzhak Mandelbaum   return (isEmptyOptional(Val1, Env1) && isEmptyOptional(Val2, Env2))
941c0725865SYitzhak Mandelbaum              ? ComparisonResult::Same
942c0725865SYitzhak Mandelbaum              : ComparisonResult::Different;
9438fcdd625SStanislav Gatev }
9448fcdd625SStanislav Gatev 
9458fcdd625SStanislav Gatev bool UncheckedOptionalAccessModel::merge(QualType Type, const Value &Val1,
9468fcdd625SStanislav Gatev                                          const Environment &Env1,
9478fcdd625SStanislav Gatev                                          const Value &Val2,
9488fcdd625SStanislav Gatev                                          const Environment &Env2,
9498fcdd625SStanislav Gatev                                          Value &MergedVal,
9508fcdd625SStanislav Gatev                                          Environment &MergedEnv) {
951c0725865SYitzhak Mandelbaum   if (!isOptionalType(Type))
9528fcdd625SStanislav Gatev     return true;
953d34fbf2dSYitzhak Mandelbaum   // FIXME: uses same approach as join for `BoolValues`. Requires non-const
954d34fbf2dSYitzhak Mandelbaum   // values, though, so will require updating the interface.
9558fcdd625SStanislav Gatev   auto &HasValueVal = MergedEnv.makeAtomicBoolValue();
956d34fbf2dSYitzhak Mandelbaum   bool MustNonEmpty1 = isNonEmptyOptional(Val1, Env1);
957d34fbf2dSYitzhak Mandelbaum   bool MustNonEmpty2 = isNonEmptyOptional(Val2, Env2);
958d34fbf2dSYitzhak Mandelbaum   if (MustNonEmpty1 && MustNonEmpty2)
9598fcdd625SStanislav Gatev     MergedEnv.addToFlowCondition(HasValueVal);
960d34fbf2dSYitzhak Mandelbaum   else if (
961d34fbf2dSYitzhak Mandelbaum       // Only make the costly calls to `isEmptyOptional` if we got "unknown"
962d34fbf2dSYitzhak Mandelbaum       // (false) for both calls to `isNonEmptyOptional`.
963d34fbf2dSYitzhak Mandelbaum       !MustNonEmpty1 && !MustNonEmpty2 && isEmptyOptional(Val1, Env1) &&
964d34fbf2dSYitzhak Mandelbaum       isEmptyOptional(Val2, Env2))
9658fcdd625SStanislav Gatev     MergedEnv.addToFlowCondition(MergedEnv.makeNot(HasValueVal));
9668fcdd625SStanislav Gatev   setHasValue(MergedVal, HasValueVal);
9678fcdd625SStanislav Gatev   return true;
9688fcdd625SStanislav Gatev }
9698fcdd625SStanislav Gatev 
970d34fbf2dSYitzhak Mandelbaum Value *UncheckedOptionalAccessModel::widen(QualType Type, Value &Prev,
971d34fbf2dSYitzhak Mandelbaum                                            const Environment &PrevEnv,
972d34fbf2dSYitzhak Mandelbaum                                            Value &Current,
973d34fbf2dSYitzhak Mandelbaum                                            Environment &CurrentEnv) {
974d34fbf2dSYitzhak Mandelbaum   switch (compare(Type, Prev, PrevEnv, Current, CurrentEnv)) {
975d34fbf2dSYitzhak Mandelbaum   case ComparisonResult::Same:
976d34fbf2dSYitzhak Mandelbaum     return &Prev;
977d34fbf2dSYitzhak Mandelbaum   case ComparisonResult::Different:
978d34fbf2dSYitzhak Mandelbaum     if (auto *PrevHasVal =
979d34fbf2dSYitzhak Mandelbaum             cast_or_null<BoolValue>(Prev.getProperty("has_value"))) {
980d34fbf2dSYitzhak Mandelbaum       if (isa<TopBoolValue>(PrevHasVal))
981d34fbf2dSYitzhak Mandelbaum         return &Prev;
982d34fbf2dSYitzhak Mandelbaum     }
983d34fbf2dSYitzhak Mandelbaum     if (auto *CurrentHasVal =
984d34fbf2dSYitzhak Mandelbaum             cast_or_null<BoolValue>(Current.getProperty("has_value"))) {
985d34fbf2dSYitzhak Mandelbaum       if (isa<TopBoolValue>(CurrentHasVal))
986d34fbf2dSYitzhak Mandelbaum         return &Current;
987d34fbf2dSYitzhak Mandelbaum     }
988d34fbf2dSYitzhak Mandelbaum     return &createOptionalValue(CurrentEnv, CurrentEnv.makeTopBoolValue());
989d34fbf2dSYitzhak Mandelbaum   case ComparisonResult::Unknown:
990d34fbf2dSYitzhak Mandelbaum     return nullptr;
991d34fbf2dSYitzhak Mandelbaum   }
992d34fbf2dSYitzhak Mandelbaum   llvm_unreachable("all cases covered in switch");
993d34fbf2dSYitzhak Mandelbaum }
994d34fbf2dSYitzhak Mandelbaum 
99558fe7f96SSam Estep UncheckedOptionalAccessDiagnoser::UncheckedOptionalAccessDiagnoser(
99658fe7f96SSam Estep     UncheckedOptionalAccessModelOptions Options)
99758fe7f96SSam Estep     : DiagnoseMatchSwitch(buildDiagnoseMatchSwitch(Options)) {}
99858fe7f96SSam Estep 
99958fe7f96SSam Estep std::vector<SourceLocation> UncheckedOptionalAccessDiagnoser::diagnose(
10007538b360SWei Yi Tee     ASTContext &Ctx, const CFGElement *Elt, const Environment &Env) {
10017538b360SWei Yi Tee   return DiagnoseMatchSwitch(*Elt, Ctx, Env);
100258fe7f96SSam Estep }
100358fe7f96SSam Estep 
1004af98b0afSStanislav Gatev } // namespace dataflow
1005af98b0afSStanislav Gatev } // namespace clang
1006