[PATCH] D62890: [DAGCombiner] Improve tryStoreMergeOfExtracts by using double sized vector type before type legalized
Simon Pilgrim via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 1 05:12:34 PDT 2022
RKSimon added inline comments.
================
Comment at: llvm/test/CodeGen/PowerPC/extract-and-store.ll:733
+; CHECK-NEXT: xxswapd vs0, vs34
+; CHECK-NEXT: stfdx f0, 0, r5
; CHECK-NEXT: blr
----------------
tingwang wrote:
> RKSimon wrote:
> > is this really an issue with the store merging or the ppc shuffle combines have gotten messed up?
> Had a quick check, this is the case PPC::LowerVECTOR_SHUFFLE does not have efficient solution, so it turned into VPERM as last resort.
>
> Probably the cost function should avoid this kind of situation.
I'd be nervous about a cost function as that is likely to be very difficult to keep balanced. I'd probably recommend just overriding canMergeStoresTo or isMultiStoresCheaperThanBitsMerge for PPC
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D62890/new/
https://reviews.llvm.org/D62890
More information about the llvm-commits
mailing list