[PATCH] D114916: [LoopInterchange] Enable loop interchange with multiple outer loop indvars

[Congzhe Cao via Phabricator via llvm-commits]

congzhe added inline comments.


================
Comment at: llvm/test/Transforms/LoopInterchange/interchangeable-outerloop-multiple-indvars.ll:13
+; void fn1() {
+;  for (; c && e; c++,e++) {
+;    d = 5;
----------------
bmahjour wrote:
> can these tests be changed to something more canonical (eg not relying on zero initialization of globals, and signed integer wrap for termination)?
Thanks for your comment! While working on the revision, can I clarify that regarding "signed integer wrap for termination", did you suggest to remove those nsw flags that are involved in the loop exit condition?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114916