[PATCH] D62890: [DAGCombiner] Improve tryStoreMergeOfExtracts to merge stores before type is legalized

Ting Wang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 8 22:43:36 PDT 2022


tingwang updated this revision to Diff 451048.
tingwang retitled this revision from "[DAGCombiner] Improve tryStoreMergeOfExtracts by using double sized vector type before type legalized " to "[DAGCombiner] Improve tryStoreMergeOfExtracts to merge stores before type is legalized".
tingwang edited the summary of this revision.
tingwang added a comment.

The original approach is too complex, and the same effect can be achieved more simply as Nemanja pointed out.

I'm adopting the whole approach, and added a guard to make sure this is applied only before type is legalized.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D62890

Files:
  llvm/include/llvm/CodeGen/TargetLowering.h
  llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
  llvm/lib/Target/PowerPC/PPCISelLowering.cpp
  llvm/lib/Target/PowerPC/PPCISelLowering.h
  llvm/test/CodeGen/PowerPC/extract-and-store.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D62890.451048.patch
Type: text/x-patch
Size: 14851 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220809/c3cd4e7d/attachment.bin>


More information about the llvm-commits mailing list