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

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert added a comment.

I feel there are opportunities here as our vector handling needs to improve a lot.
Though, without more data to decide (1) where to put this (in the pipeline) and (2) when to enable this (platform, context,...), it is hard to say for sure.
Generally, I would like to have this in-tree (potentially under a flag).

I think the code looks good, though we can add FIXMEs for missing parts, e.g, better alignment, the load version of this, ...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71828