[PATCH] D78990: [analyzer] Allow bindings of the CompoundLiteralRegion

Artem Dergachev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 29 03:11:07 PDT 2020


NoQ added a comment.

Yay unit tests! Let's squash those mutually cancelling bugs.



================
Comment at: clang/unittests/StaticAnalyzer/StoreTest.cpp:22-32
+  struct TestContext {
+    StoreManager &SManager;
+    SValBuilder &Builder;
+    MemRegionManager &MRManager;
+    ASTContext &ASTContext;
+
+    /* implicit */ TestContext(ExprEngine &Engine)
----------------
Why was this necessary? All these variables seem to be already available in the surrounding scope as fields in `ExprEngineConsumer`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78990





More information about the cfe-commits mailing list