[PATCH] D84399: [SCEVExpander] Avoid re-sing existing casts if it means updating users.

[Eli Friedman via Phabricator via llvm-commits]

efriedma added inline comments.


================
Comment at: llvm/test/Transforms/LoopDistribute/bounds-expansion-bug.ll:8
+; should only be a small optimization and can be re-introduced once we can
+; properly clean up expansion results, if they are unused.
+
----------------
Does the pass semantically depend on reusing casts, or does failing to reuse casts just make the generated code uglier?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84399