[PATCH] D70223: [DAGCombine] Split vector load-update-store into single element stores
Qiu Chaofan via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Nov 28 02:18:17 PST 2019
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>
More information about the llvm-commits
mailing list