[PATCH] D98240: [VectorCombine] Simplify to scalar store if only one element updated

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added a comment.

Hmm wait, i completely ignored this patch :/
Does this really not do any cost modelling?
This should at least check that scalar load isn't more costly than the original vector load.



================
Comment at: llvm/lib/Transforms/Vectorize/VectorCombine.cpp:768-770
+  return std::any_of(Begin, End, [&](const Instruction &Instr) {
+    return isModSet(AA.getModRefInfo(&Instr, Loc)) ||
+           ++NumScanned > MaxInstrsToScan;
----------------
Does this ignore debuginfo?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98240