[PATCH] D103700: [LV] Fix bug when unrolling (only) a loop with non-latch exit

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 11 11:25:20 PDT 2021


reames added a comment.

@Ayal If I'm understanding you even partially correctly, it sounds like you're raising a code quality issue.  That is, we may generate a dead epilogue loop (e.g. with a condition statically known to result in the epilogue being untaken, but emitted as a condition) when we didn't need to.  Is that correct?

If so, I request that we land this patch - which is fixing a functional bug - and handle the code quality issue in a separate patch.

I really don't understand the interweaving code, and would really rather not have to.  :)


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

https://reviews.llvm.org/D103700



More information about the llvm-commits mailing list