[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