[clang][dataflow] Improve optional model's support for ignoring smart pointers.
authorYitzhak Mandelbaum <yitzhakm@google.com>
Wed, 14 Dec 2022 13:54:38 +0000 (13:54 +0000)
committerYitzhak Mandelbaum <yitzhakm@google.com>
Thu, 15 Dec 2022 15:39:52 +0000 (15:39 +0000)
commit5d22d1f54836263ead1971ef8128f5128290dfa7
tree5737a1f62d1171b46fc8e50b425b5f004190827d
parent07343dc0b6a29e23a93bc16fa8d3fa9c584851c1
[clang][dataflow] Improve optional model's support for ignoring smart pointers.

The optional model has an option to ignore optionals accessed through smart
pointer types (other than optional itself). This patch improves this feature in
two ways:

1. We extend support to optionals accessed directly through the smart pointer,
like `ptr->value()`. Previously, support was limited to cases that went through
an intermediate field.

2. We clean up the implementation by removing the option from the analysis,
leaving it only in the diagnostic phase (where it is relevant).

3. Adjusts a test which was misleading in what it was testing.

Differential Revision: https://reviews.llvm.org/D140020
clang/include/clang/Analysis/FlowSensitive/Models/UncheckedOptionalAccessModel.h
clang/lib/Analysis/FlowSensitive/Models/UncheckedOptionalAccessModel.cpp
clang/unittests/Analysis/FlowSensitive/UncheckedOptionalAccessModelTest.cpp