[PATCH] D37400: [StaticAnalyzer] Fix failures due to the iteration order of ExplodedNode

Mandeep Singh Grang via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 6 15:52:59 PDT 2017


mgrang added inline comments.


================
Comment at: include/clang/StaticAnalyzer/Core/PathSensitive/ExplodedGraph.h:407
 class ExplodedNodeSet {
-  typedef llvm::SmallPtrSet<ExplodedNode*,5> ImplTy;
+  typedef llvm::SmallSetVector<ExplodedNode*, 4> ImplTy;
   ImplTy Impl;
----------------
SmallSetVector size has to be a power of 2. I was split between 4 and 8. I chose 4 as it's closer to 5 :)


https://reviews.llvm.org/D37400





More information about the cfe-commits mailing list