[PATCH] D33837: Check the loop header before updating it in UpdateAnalysisInformation

Anna Thomas via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 2 09:25:33 PDT 2017


anna added a comment.

One thing to mention: I think currently there's no case where `SplitMakesNewLoopHeader` is true when the `OldBB` is not a header in `UpdateAnalysisInformation`. This happens only in IRCE. However, it is possible at some point, when a subloop does not have dedicated exits, and we incorrectly update the header of the subloop.


https://reviews.llvm.org/D33837





More information about the llvm-commits mailing list