From af22be39038a9c464474410c3a8b3cb428d8b25a Mon Sep 17 00:00:00 2001 From: Martin Braenne Date: Mon, 5 Jun 2023 11:30:39 +0000 Subject: [PATCH] [clang][dataflow] Use a `PointerValue` for `value` property in optional checker. The `ReferenceValue` class will be eliminated as part of the ongoing migration to strict handling of value categories (see https://discourse.llvm.org/t/70086 for details). Reviewed By: gribozavr2 Differential Revision: https://reviews.llvm.org/D152144 --- .../FlowSensitive/Models/UncheckedOptionalAccessModel.cpp | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/clang/lib/Analysis/FlowSensitive/Models/UncheckedOptionalAccessModel.cpp b/clang/lib/Analysis/FlowSensitive/Models/UncheckedOptionalAccessModel.cpp index 7d30473..1095fd4 100644 --- a/clang/lib/Analysis/FlowSensitive/Models/UncheckedOptionalAccessModel.cpp +++ b/clang/lib/Analysis/FlowSensitive/Models/UncheckedOptionalAccessModel.cpp @@ -307,12 +307,12 @@ StorageLocation *maybeInitializeOptionalValueMember(QualType Q, Environment &Env) { // The "value" property represents a synthetic field. As such, it needs // `StorageLocation`, like normal fields (and other variables). So, we model - // it with a `ReferenceValue`, since that includes a storage location. Once + // it with a `PointerValue`, since that includes a storage location. Once // the property is set, it will be shared by all environments that access the // `Value` representing the optional (here, `OptionalVal`). if (auto *ValueProp = OptionalVal.getProperty("value")) { - auto *ValueRef = clang::cast(ValueProp); - auto &ValueLoc = ValueRef->getReferentLoc(); + auto *ValuePtr = clang::cast(ValueProp); + auto &ValueLoc = ValuePtr->getPointeeLoc(); if (Env.getValue(ValueLoc) == nullptr) { // The property was previously set, but the value has been lost. This can // happen, for example, because of an environment merge (where the two @@ -339,8 +339,8 @@ StorageLocation *maybeInitializeOptionalValueMember(QualType Q, return nullptr; auto &ValueLoc = Env.createStorageLocation(Ty); Env.setValue(ValueLoc, *ValueVal); - auto &ValueRef = Env.create(ValueLoc); - OptionalVal.setProperty("value", ValueRef); + auto &ValuePtr = Env.create(ValueLoc); + OptionalVal.setProperty("value", ValuePtr); return &ValueLoc; } -- 2.7.4