[PATCH] D46044: [LoopUnrollPeel] Fix potentially incorrect invalidation of SCEV in peelLoop

Max Kazantsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 6 23:39:30 PST 2019


mkazantsev added a comment.

Hi Florian,

If you take a look at `computeBackedgeTakenCount` method, it constructs `BackedgeTakenInfo` that contains references on loop's exiting blocks inside. This info is then cached in `BackedgeTakenCounts`. If we change an inner loop so that one of these blocks gets deleted or stops being exiting, any of its parents may end up keeping dangling pointers to this block (because exiting block of inner loop can also be an exiting block of its parents).


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

https://reviews.llvm.org/D46044





More information about the llvm-commits mailing list