[PATCH] D110234: [LoopFlatten] Updating Phi nodes after IV widening

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 27 00:07:04 PDT 2021


dmgreen added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/LoopFlatten.cpp:106
+      return false;
+    return InnerPHIsToTransform.count(Phi) || NarrowOuterInductionPHI == Phi;
+  }
----------------
Other places add to InnerInductionPHI. Is it always safe (now and in the future) to assume they are all narrow induction phis?


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

https://reviews.llvm.org/D110234



More information about the llvm-commits mailing list