[PATCH] D121378: [clang][dataflow] Model the behavior of various optional members

2022-03-13 Thread Stanislav Gatev via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. sgatev marked an inline comment as done. Closed by commit rG9e0fc6768378: [clang][dataflow] Model the behavior of various optional members (authored by sgatev). Reposi

[PATCH] D121378: [clang][dataflow] Model the behavior of various optional members

2022-03-10 Thread Stanislav Gatev via Phabricator via cfe-commits
sgatev marked an inline comment as done. sgatev added inline comments. Comment at: clang/unittests/Analysis/FlowSensitive/UncheckedOptionalAccessModelTest.cpp:33 static constexpr char StdTypeTraitsHeader[] = R"( +#ifndef TYPE_TRAITS_H +#define TYPE_TRAITS_H xaz

[PATCH] D121378: [clang][dataflow] Model the behavior of various optional members

2022-03-10 Thread Stanislav Gatev via Phabricator via cfe-commits
sgatev updated this revision to Diff 414396. sgatev marked an inline comment as done. sgatev added a comment. Address reviewers' comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121378/new/ https://reviews.llvm.org/D121378 Files: clang/li

[PATCH] D121378: [clang][dataflow] Model the behavior of various optional members

2022-03-10 Thread Gábor Horváth via Phabricator via cfe-commits
xazax.hun accepted this revision. xazax.hun added a comment. This revision is now accepted and ready to land. Thanks! Comment at: clang/lib/Analysis/FlowSensitive/Models/UncheckedOptionalAccessModel.cpp:64 static BoolValue *getHasValue(Value *Val) { - if (auto *OptionalVal =

[PATCH] D121378: [clang][dataflow] Model the behavior of various optional members

2022-03-10 Thread Stanislav Gatev via Phabricator via cfe-commits
sgatev created this revision. sgatev added reviewers: ymandel, xazax.hun, gribozavr2. Herald added subscribers: tschuett, steakhal, rnkovacs. Herald added a project: All. sgatev requested review of this revision. Herald added a project: clang. Model `make_optional`, optional's default constructor,