[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 03:26:15 PDT 2022


RKSimon added a comment.

Instead of a cost function, could we use (possibly tweaked) isMultiStoresCheaperThanBitsMerge?



================
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
----------------
is this really an issue with the store merging or the ppc shuffle combines have gotten messed up?


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