[cfe-commits] r74465 - in /cfe/trunk: include/clang/AST/DeclCXX.h include/clang/Basic/DiagnosticSemaKinds.td lib/Sema/SemaDeclCXX.cpp test/SemaCXX/class-base-member-init.cpp

Fariborz Jahanian fjahanian at apple.com
Mon Jun 29 17:19:46 PDT 2009


Done in http://llvm.org/viewvc/llvm-project?view=rev&revision=74486
Except for one. See below.


On Jun 29, 2009, at 3:41 PM, Chris Lattner wrote:
>>
>> +  llvm::DenseSet<uintptr_t>Members;
>
> Using void* would also allow use of SmallPtrSet<void*, 16> here,  
> which avoids going to the heap in the common case.  DenseSet always  
> goes to to heap.

I am now using DenseMap which presumably still goes to the heap.  
Please see its use and see if there is a stack based container
which does the same.

- Thanks, Fariborz




More information about the cfe-commits mailing list