This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rG37e7673c21af: [PowerPC] Implement Move to VSR Mask builtins
in LLVM/Clang (authored by amyk).
Changed prior to commit:
steven.zhang accepted this revision.
steven.zhang added a comment.
This revision is now accepted and ready to land.
LGTM.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82725/new/
https://reviews.llvm.org/D82725
amyk updated this revision to Diff 292634.
amyk added a comment.
- Rebased patch.
- Update patch to remove unnecessary immediate handling.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82725/new/
https://reviews.llvm.org/D82725
Files:
amyk added inline comments.
Comment at: llvm/lib/Target/PowerPC/PPCISelLowering.cpp:10054
+
+ case Intrinsic::ppc_altivec_mtvsrbm: {
+// The llvm.ppc.altivec.mtvsrbm intrinsic can correspond to two different
steven.zhang wrote:
> Can we handle this inside
steven.zhang added inline comments.
Comment at: llvm/lib/Target/PowerPC/PPCISelLowering.cpp:10054
+
+ case Intrinsic::ppc_altivec_mtvsrbm: {
+// The llvm.ppc.altivec.mtvsrbm intrinsic can correspond to two different
Can we handle this inside the .td ? i.e.
amyk added a comment.
Ping.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82725/new/
https://reviews.llvm.org/D82725
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
amyk added a comment.
Ping.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82725/new/
https://reviews.llvm.org/D82725
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
amyk updated this revision to Diff 283113.
amyk added a comment.
Rebased patch, and removed MC tests from the original patch.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82725/new/
https://reviews.llvm.org/D82725
Files: