[PATCH] Add a 'covers' operation to LiveRange

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


And another one with an actually new patch…

-------------- next part --------------
A non-text attachment was scrubbed...
Name: Add-a-covers-operation-to-LiveRange.patch
Type: application/octet-stream
Size: 2097 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131007/f5e88376/attachment.obj>
-------------- next part --------------

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

> And another version without typos.
> 
> Greetings
> 	Matthias
> 
> <Add-a-covers-operation-to-LiveRange.patch>
> 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
> 
> _______________________________________________
> 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