[PATCH] D141778: [DAGCombiner][X86] `mergeConsecutiveStores()`: support merging splat-stores of the same value

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jan 15 06:56:17 PST 2023


lebedev.ri added a comment.

In D141778#4054600 <https://reviews.llvm.org/D141778#4054600>, @lebedev.ri wrote:

> Add subvector test coverage. 
> @spatel should we not split such stores just like we do with stores of CONCAT_VECTOR?

Ah, so we already do that in `collectConcatOps()`. Okay, then these must be improvements.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D141778



More information about the llvm-commits mailing list