[PATCH] Add a 'covers' operation to LiveRange

Matthias Braun mbraun at apple.com
Mon Oct 7 15:12:05 PDT 2013


And another version without typos.

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/c7d52c64/attachment.obj>
-------------- next part --------------

On Oct 7, 2013, at 3:02 PM, Matthias Braun <mbraun at apple.com> wrote:

> 
> 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
> 
> <Add-a-covers-operation-to-LiveRange.patch>_______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits



More information about the llvm-commits mailing list