Refactor ThreadSafety to use PostOrderCFGView instead of its own copy (of TopologicallySortedCFG).

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@142714 91177308-0d34-0410-b5e6-96231b3b80d8

1 files changed, 8 insertions, 0 deletions

diff --git a/include/clang/Analysis/Analyses/PostOrderCFGView.h b/include/clang/Analysis/Analyses/PostOrderCFGView.h
index b3a027288a..0891343312 100644
--- a/include/clang/Analysis/Analyses/PostOrderCFGView.h
+++ b/include/clang/Analysis/Analyses/PostOrderCFGView.h
@@ -27,6 +27,13 @@
 namespace clang {
     
 class PostOrderCFGView : public ManagedAnalysis {
+public:
+  /// \brief Implements a set of CFGBlocks using a BitVector.
+  ///
+  /// This class contains a minimal interface, primarily dictated by the SetType
+  /// template parameter of the llvm::po_iterator template, as used with external
+  /// storage. We also use this set to keep track of which CFGBlocks we visit
+  /// during the analysis.
   class CFGBlockSet {
     llvm::BitVector VisitedBlockIDs;
   public:
@@ -60,6 +67,7 @@ class PostOrderCFGView : public ManagedAnalysis {
     }
   };
 
+private:
   typedef llvm::po_iterator<const CFG*, CFGBlockSet, true>  po_iterator;  
   std::vector<const CFGBlock*> Blocks;