[PATCH] D37870: Refactor collectChildrenInLoop to LoopUtils [NFC]

[Alina Sbirlea via Phabricator via llvm-commits]

asbirlea marked 2 inline comments as done.
asbirlea added inline comments.


================
Comment at: lib/Transforms/Utils/LoopUtils.cpp:1133
+
+  for (size_t I = 0; I < Worklist.size(); I++) {
+    DomTreeNode *DTN = Worklist[I];
----------------
sanjoy wrote:
> For integer loop counters, I've usually seem LLVM use `i` instead of `I`.
That's true. I've seen both. Keeping I per the variable style guide.


https://reviews.llvm.org/D37870