[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