[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