[PATCH] D101305: [LoopInterchange] Fix legality for triangular loops

[Congzhe Cao via Phabricator via llvm-commits]

congzhe added inline comments.


================
Comment at: llvm/test/Transforms/LoopInterchange/inner-indvar-depend-on-outer-indvar.ll:24
+  %i = phi i64 [ 0, %entry ], [ %i.next, %for1.inc10 ]
+  %upperbound = load i64, i64* @B
+  br label %for2
----------------
bmahjour wrote:
> I don't see this being used!
Thanks, updated all tests accordingly.
This `%upperbound` is a leftover when I was experimenting the non-constant upper bound you mentioned. Now deleted it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101305