[polly] Little refactoring

Tobias Grosser tobias at grosser.es
Fri Jun 13 09:40:21 PDT 2014


On 13/06/2014 18:08, Johannes Doerfert wrote:
>>> On 06/13/2014 01:27 AM, Johannes Doerfert wrote:
>>>> Refactoring:
>>>>
>>>> -          Refactored reoccurring if cascade
>>>
>>> LGTM.
> One more to commit.
>
>>>> -          C++11 iterators & range loops.
>>>
>>> I'm not sure what others think, but I don't like range loops that use
> auto for the element type. It is a MemoryAccess, why not call it that way
> ;-).
> I can do both [As long as I can use range loops ;)]. Is there a llvm
> guideline? If not let's define one for us.

I would use the explicit type, except if this type is obviously 
redundant or overly complex.

>>> Apart from that, LGTM.
> Once the above is cleared, one more to commit.

Feel free to commit. (Just reply with the commit id, such that it 
becomes clear this thread is closed)

Tobias



More information about the llvm-commits mailing list