[PATCH] D107964: [LoopIdiom] Don't transform loop into memmove when load from body has more than one use

[Eli Friedman via Phabricator via llvm-commits]

efriedma added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/LoopIdiomRecognize.cpp:1314
+                            StoreSizeSCEV, *AA, Stores) ||
+      (UseMemMove && !TheLoad->hasOneUse())) {
     ORE.emit([&]() {
----------------
I think this belongs earlier, with the other mayLoopAccessLocation check?  It's basically a variation of the same check.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107964