[cfe-commits] r84607 - /cfe/trunk/include/clang/Analysis/Support/BumpVector.h

Ted Kremenek kremenek at apple.com
Mon Oct 19 22:49:18 PDT 2009


Author: kremenek
Date: Tue Oct 20 00:49:17 2009
New Revision: 84607

URL: http://llvm.org/viewvc/llvm-project?rev=84607&view=rev
Log:
BumpVectorContext: Use 'unsigned' integer type with PointerIntUnion instead of bool to keep it clear that we are reasoning about an unsigned integer with a single bit.

Modified:
    cfe/trunk/include/clang/Analysis/Support/BumpVector.h

Modified: cfe/trunk/include/clang/Analysis/Support/BumpVector.h
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Analysis/Support/BumpVector.h?rev=84607&r1=84606&r2=84607&view=diff

==============================================================================
--- cfe/trunk/include/clang/Analysis/Support/BumpVector.h (original)
+++ cfe/trunk/include/clang/Analysis/Support/BumpVector.h Tue Oct 20 00:49:17 2009
@@ -27,16 +27,16 @@
 namespace clang {
   
 class BumpVectorContext {
-  llvm::PointerIntPair<llvm::BumpPtrAllocator*, 1, bool> Alloc;
+  llvm::PointerIntPair<llvm::BumpPtrAllocator*, 1> Alloc;
 public:
   /// Construct a new BumpVectorContext that creates a new BumpPtrAllocator
   /// and destroys it when the BumpVectorContext object is destroyed.
-  BumpVectorContext() : Alloc(new llvm::BumpPtrAllocator(), true) {}
+  BumpVectorContext() : Alloc(new llvm::BumpPtrAllocator(), 1) {}
   
   /// Construct a new BumpVectorContext that reuses an existing
   /// BumpPtrAllocator.  This BumpPtrAllocator is not destroyed when the
   /// BumpVectorContext object is destroyed.
-  BumpVectorContext(llvm::BumpPtrAllocator &A) : Alloc(&A, false) {}
+  BumpVectorContext(llvm::BumpPtrAllocator &A) : Alloc(&A, 0) {}
   
   ~BumpVectorContext() {
     if (Alloc.getInt())





More information about the cfe-commits mailing list