[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