[llvm-commits] [llvm] r122680 - in /llvm/trunk/include/llvm/Analysis: DominatorInternals.h Dominators.h

[Cameron Zwarich]

On Jan 3, 2011, at 12:37 AM, Duncan Sands wrote:

> Hi Cameron,
> 
>> --- llvm/trunk/include/llvm/Analysis/DominatorInternals.h (original)
>> +++ llvm/trunk/include/llvm/Analysis/DominatorInternals.h Sun Jan  2 01:03:00 2011
>> @@ -257,12 +257,24 @@
>>    // infinite loops). In these cases an artificial exit node is required.
>>    MultipleRoots |= (DT.isPostDominator()&&  N != F.size());
>> 
>> +  std::vector<unsigned>  Buckets;
> 
> how about using SmallVector?

That might be a good idea. I guess I didn't think about it because there is already so much other allocation going on, and this is just once per pass.

Cameron