[PATCH] Add range for-loop iterator adapter for cases in SwitchInst

Marcello Maggioni hayarms at gmail.com
Tue Jul 1 14:32:25 PDT 2014


Thanks for the review :)

Needs somebody to commit it if nobody has any other concerns about it.

Marcello


2014-07-01 22:29 GMT+02:00 Reid Kleckner <rnk at google.com>:

> lgtm
>
>
> On Tue, Jul 1, 2014 at 7:28 AM, Marcello Maggioni <hayarms at gmail.com>
> wrote:
>
>> Bump for this
>>
>>
>> 2014-06-27 16:42 GMT+02:00 Marcello Maggioni <hayarms at gmail.com>:
>>
>> Hi, after some suggestions about adding a range for-loop adapter for
>>> SwitchInst cases iterator I made this patch to add them.
>>>
>>> I needed to add kind of a "fake" operator* to CaseIt , because the Case
>>> iterator for switches is more than an iterator, but it is also basically
>>> the iterated object itself.
>>>
>>> Needs review and eventual commit (I don't have commit access :) )
>>>
>>> Marcello
>>>
>>
>>
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140701/c8ff278b/attachment.html>


More information about the llvm-commits mailing list