[PATCH] D75059: [PowerPC][NFC] Remove support for VRSAVE save/restore/update.

Sean Fertile via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 31 10:26:21 PDT 2020


sfertile updated this revision to Diff 288981.
sfertile added a comment.

Previous patch was not really NFC, since we were inserting VRSAVE pseudos on AIX, then emiting a fatal error in frame lowering. This was because we missed changing the condition gating the vrsave insertion to include the AIX abi.

I've landed a patch which shows the previous error related to VRSAVE, and the new error I've added in LowerReturn to replace it. The new behavior is more lenient in that we can use vector instructions inside a functions body, and will now only error if we try to pass or return vector types.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75059

Files:
  llvm/lib/Target/PowerPC/PPCFrameLowering.cpp
  llvm/lib/Target/PowerPC/PPCISelDAGToDAG.cpp
  llvm/lib/Target/PowerPC/PPCISelLowering.cpp
  llvm/lib/Target/PowerPC/PPCInstrInfo.cpp
  llvm/lib/Target/PowerPC/PPCInstrInfo.h
  llvm/lib/Target/PowerPC/PPCInstrInfo.td
  llvm/lib/Target/PowerPC/PPCMachineFunctionInfo.h
  llvm/lib/Target/PowerPC/PPCRegisterInfo.cpp
  llvm/lib/Target/PowerPC/PPCRegisterInfo.h
  llvm/lib/Target/PowerPC/README_ALTIVEC.txt
  llvm/test/CodeGen/PowerPC/aix-vector-return.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D75059.288981.patch
Type: text/x-patch
Size: 25789 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200831/984277e8/attachment.bin>


More information about the llvm-commits mailing list