[PATCH] D71828: [InstCombine] Convert vector store to scalar store if only one element updated

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

One thing I've realized reading this thread again is that it's not only the compiler that can get confused by a wrong-width store; the CPU itself can also run into issues with store->load forwarding.  See recent discussion http://lists.llvm.org/pipermail/llvm-dev/2020-May/141837.html .  So whether this transform is worthwhile might depend on the context.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71828