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

Ashutosh Nema via llvm-commits llvm-commits at lists.llvm.org
Sun Apr 17 22:42:07 PDT 2016


ashutosh.nema added a comment.

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 ?


http://reviews.llvm.org/D15922





More information about the llvm-commits mailing list