This is based on the patch in https://reviews.llvm.org/D106353 provided by @nemanjai
Add more splat load types handling and more tests.
Paths
| Differential D106555
[PowerPC] handle more splat loads ClosedPublic Authored by shchenz on Jul 22 2021, 8:15 AM.
Details
Summary This is based on the patch in https://reviews.llvm.org/D106353 provided by @nemanjai Add more splat load types handling and more tests.
Diff Detail
Unit TestsFailed Event Timelineshchenz mentioned this in D106357: [PowerPC] custom v2i64 for scalar_to_vector.Jul 26 2021, 12:00 AM
shchenz marked an inline comment as done. Comment Actions1: Lint warning fix shchenz added inline comments.
Comment Actions @nemanjai Hi Nemanja, could you please help to have another look at this issue? Thanks.
shchenz marked 8 inline comments as done. Comment Actionsaddress @jsji comments
Comment Actions LGTM. Thanks.
This revision is now accepted and ready to land.Nov 1 2021, 8:45 AM This revision was landed with ongoing or failed builds.Nov 2 2021, 10:33 PM Closed by commit rG5a8b19634002: [PowerPC] handle more splat loads without stack operation (authored by shchenz). · Explain Why This revision was automatically updated to reflect the committed changes.
Comment Actions Thanks for your comments @nemanjai I addressed most of the comment in commit 96950270669acd3c342a266562ff3a41464cc0a0 One comment related to the code gen improvement for the aligned load will be addressed in the follow-up Phabricator patch.
Revision Contents
Diff 360822 llvm/lib/Target/PowerPC/PPCISelLowering.h
llvm/lib/Target/PowerPC/PPCISelLowering.cpp
llvm/lib/Target/PowerPC/PPCInstrVSX.td
llvm/lib/Target/PowerPC/PPCMIPeephole.cpp
llvm/test/CodeGen/PowerPC/canonical-merge-shuffles.ll
llvm/test/CodeGen/PowerPC/load-and-splat.ll
llvm/test/CodeGen/PowerPC/scalar_vector_test_3.ll
|
clang-format: please reformat the code