[PATCH] D114978: [PowerPC] Fix load/store selection infrastructure when load/store intrinsics are used on P10.

Amy Kwan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 2 11:37:29 PST 2021


amyk created this revision.
amyk added reviewers: nemanjai, lei, stefanp, PowerPC.
amyk added projects: PowerPC, LLVM.
Herald added subscribers: shchenz, kbarton, hiraditya.
amyk requested review of this revision.

The load/store infrastructure previously made an incorrect assumption that whenever it is used with 
a load/store intrinsic on Power10 - those intrinsics would automatically be the lxvp/stxvp intrinsics
introduced in Power10. However, this is obviously not the case as there are multiple instances of
pre-P10 intrinsics that use the refactored load/store implementation.

This patch corrects this assumption, and produces the expected intrinsic on pre-P10.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D114978

Files:
  llvm/lib/Target/PowerPC/PPCISelLowering.cpp
  llvm/test/CodeGen/PowerPC/lxvw4x-bug.ll
  llvm/test/CodeGen/PowerPC/swaps-le-8.ll
  llvm/test/CodeGen/PowerPC/vsx-ldst-builtin-le.ll
  llvm/test/CodeGen/PowerPC/vsx-p9.ll
  llvm/test/CodeGen/PowerPC/vsx_builtins.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D114978.391400.patch
Type: text/x-patch
Size: 10564 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20211202/4b8750c5/attachment-0001.bin>


More information about the llvm-commits mailing list