[PATCH] Add a 'covers' operation to LiveRange
    Matthias Braun 
    mbraun at apple.com
       
    Mon Oct  7 15:02:18 PDT 2013
    
    
  
On Oct 7, 2013, at 2:34 PM, Andrew Trick <atrick at apple.com> wrote:
> 
> On Oct 7, 2013, at 1:57 PM, Matthias Braun <mbraun at apple.com> wrote:
> 
>> <Add-a-covers-operation-to-LiveRange.patch>
> 
> +  const_iterator I = begin();
> +  for (const_iterator O = Other.begin(), OE = Other.end(); O != OE; ++O) {
> +    I = advanceTo(I, O->start);
> 
> Is it possible for advanceTo to return end() here?
Indeed. An improved version with more comments is attached.
Greetings
	Matthias
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Add-a-covers-operation-to-LiveRange.patch
Type: application/octet-stream
Size: 2100 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131007/d9579a34/attachment.obj>
    
    
More information about the llvm-commits
mailing list