[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