[PATCH] D97828: [LoopInterchange] Disallow interchange when memory accesses are guarded by control flow (PR48057)

[Ta-Wei Tu via Phabricator via llvm-commits]

TaWeiTu marked 2 inline comments as done.
TaWeiTu added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/LoopInterchange.cpp:1020
 
+  // Check if either outer and inner loop contains instructions guarded by
+  // control flow that have side effects.
----------------
Whitney wrote:
> In what scenario, outer loop contains instructions guarded by control flow?
> (I assume loop interchange operate on tight loops.)
Thanks for pointing that out! I think you are right.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97828