This revision was automatically updated to reflect the committed changes.
Closed by commit rGbd06c417e6c7: [analyzer] Allow bindings of the
CompoundLiteralRegion (authored by vsavchenko).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78990/new/
http
NoQ accepted this revision.
NoQ added a comment.
This revision is now accepted and ready to land.
Great, thanks!
Comment at: clang/unittests/StaticAnalyzer/StoreTest.cpp:66
-Store StInit = StMgr.getInitialStore(SFC).getStore();
-SVal Zero = SVB.makeZeroVal(ACtx.IntTy)
vsavchenko updated this revision to Diff 264831.
vsavchenko added a comment.
Fix code review remarks.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78990/new/
https://reviews.llvm.org/D78990
Files:
clang/lib/StaticAnalyzer/Core/RegionStore.cpp
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 &AS
vsavchenko created this revision.
vsavchenko added reviewers: NoQ, dcoughlin.
Herald added subscribers: cfe-commits, ASDenysPetrov, martong, Charusso,
dkrupp, donat.nagy, Szelethus, mikhail.ramalho, a.sidorin, szepet,
baloghadamsoftware, xazax.hun.
Herald added a project: clang.
CompoundLiteralR
