[PATCH] D15922: [Cloning] rename cloneLoopWithPreheader() and add assert to ensure no sub-loops

[Adam Nemet via llvm-commits]

anemet added a comment.

In http://reviews.llvm.org/D15922#403741, @ashutosh.nema wrote:

> Thanks Vaivaswatha, looks OK to me except the name of function.
>  As I mentioned earlier ‘cloneInnermostLoopWithPreheader’ looks more intuitive 
>  vs ‘cloneInnerLoopWithPreheader’ because we clone the innermost loop only.
>
> Adam, Hal:
>  What’s your opinion on this name change ?


Sounds good to me.


http://reviews.llvm.org/D15922