[PATCH] D56079: Fix invalid test in Guard Widening

Max Kazantsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 29 04:01:28 PST 2019


mkazantsev added a comment.

I agree that my initial assessment that it was a single loops with 2 latches was wrong. We can interpret it as 2 loops, where `inner` is an inner loop, `outer` is an outer loop and outer loop's header is created after loop canonicalization. However, even in this case, checks check that we widen the condition of guard from outer loop *into* inner loop, while the comment claims opposite.


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

https://reviews.llvm.org/D56079





More information about the llvm-commits mailing list