[LLVMdev] [PATCH] Spill Comments

[Chris Lattner]

On Sep 14, 2009, at 10:50 AM, David Greene wrote:

>>
>> Why does SpillObjects need to be a DenseSet?  It seems that it is
>> created in order.  I think it can just be a vector which is looked up
>> with a binary search.
>
> I wasn't sure ordering was guaranteed.  As I noted to Dan, I'd like  
> to get
> rid of this entirely.

I'm pretty sure it is: the "is spill slot" is a property set when the  
spill is created, and spill slots are assigned increasing numbers in  
order.

-Chris