[PATCH] D131438: [clang][dataflow] Analyze constructor bodies

Yitzhak Mandelbaum via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 9 06:14:52 PDT 2022


ymandel accepted this revision.
ymandel added inline comments.
This revision is now accepted and ready to land.


================
Comment at: clang/unittests/Analysis/FlowSensitive/TransferTest.cpp:4371
 
+TEST(TransferTest, ContextSensitiveConstructorBody) {
+  std::string Code = R"(
----------------
What about a default constructor, including when there are field initializers like:
```
 class MyClass {
    public:
      MyClass() = default;

      bool MyField = true;
    };
```

Should we expect to handle that correctly? If so, can you add some tests?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D131438/new/

https://reviews.llvm.org/D131438



More information about the cfe-commits mailing list