[PATCH] D70223: [DAGCombine] Split vector load-update-store into single element stores

[Qiu Chaofan via Phabricator via llvm-commits]

qiucf updated this revision to Diff 231380.
qiucf marked 12 inline comments as done.
qiucf added a comment.
Herald added a subscriber: jfb.

Address some comments from the community:

- Add the swap-le test back to this revision for better review.
- Add check for indexed store/loads.
- Add more tests for change-length shuffle, undef, etc.
- Subtle some comments inconsistent with code.
- Eliminate a possible infinite loop case.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70223

Files:
  llvm/include/llvm/CodeGen/TargetLowering.h
  llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
  llvm/lib/CodeGen/SelectionDAG/TargetLowering.cpp
  llvm/lib/Target/PowerPC/PPCISelLowering.cpp
  llvm/lib/Target/PowerPC/PPCISelLowering.h
  llvm/test/CodeGen/PowerPC/swaps-le-5.ll
  llvm/test/CodeGen/PowerPC/swaps-le-6.ll
  llvm/test/CodeGen/PowerPC/vector-store-split.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D70223.231380.patch
Type: text/x-patch
Size: 18476 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191128/7d9823f9/attachment.bin>