[PATCH] D35304: [RuntimeLoopUnrolling] Update DomTree correctly when exit blocks have successors

Anna Thomas via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 12 12:53:20 PDT 2017


anna added inline comments.


================
Comment at: lib/Transforms/Utils/LoopUnrollRuntime.cpp:764
+     // check for IDom being the header because successors of exit blocks can
+     // have edges from outside the loop, and we should not incorrectly dominate
+     // the IDom in that case.
----------------
nit: s/dominate/update 
Comment should read as "we should not incorrectly *update* "


https://reviews.llvm.org/D35304





More information about the llvm-commits mailing list