[PATCH] D38415: [SimplifyIndVar] Replace IVUsers with loop invariant if possible

Hongbin Zheng via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 3 10:02:48 PDT 2017


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


================
Comment at: lib/Transforms/Utils/SimplifyIndVar.cpp:553
+  // If I is the LCSSA phi, L will not contain UserL, skip I in this case
+  if (!L->contains(UserL))
+    return false;
----------------
Maybe we should not push IV users in pushIVUsers at all if IV users do not in the IV loop (e.g. LCSSA phi)


Repository:
  rL LLVM

https://reviews.llvm.org/D38415





More information about the llvm-commits mailing list