[PATCH] D154205: [MachineLICM] Handle subloops

[Wei Xiao via Phabricator via llvm-commits]

wxiao3 added inline comments.


================
Comment at: llvm/lib/CodeGen/MachineLICM.cpp:321
+  // Add loop to worklist
+  Worklist.push_back(Loop);
+
----------------
2 questions:
1) why we don't require that the outer-most loop that has a unique predecessor?
2) can we push the innermost loops into the worklist first?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D154205/new/

https://reviews.llvm.org/D154205