[PATCH] D141777: [X86] Reenable store merging post-legalization
Roman Lebedev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Jan 22 04:58:23 PST 2023
lebedev.ri added inline comments.
================
Comment at: llvm/lib/Target/X86/X86ISelLowering.h:1077
+ EVT MemVT, std::optional<StoreSource> StoreSrc) const override {
+ return !StoreSrc || StoreSrc != StoreSource::ExtractSubvector;
}
----------------
RKSimon wrote:
> should we play safe and treat unknown as a potential extractsubvector?
Let me just assert that we don't get `unknown` here.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D141777/new/
https://reviews.llvm.org/D141777
More information about the llvm-commits
mailing list