[PATCH] D133547: [PowerPC] Fix CSR_64_AllRegs_AIX_Dflt_VSX definition

Zarko Todorovski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 9 14:41:03 PDT 2022


ZarkoCA added inline comments.


================
Comment at: llvm/lib/Target/PowerPC/PPCCallingConv.td:365-366
 
-def CSR_64_AllRegs_AIX_Dflt_VSX : CalleeSavedRegs<(add CSR_64_AllRegs_Altivec,
-                                         (sequence "VSL%u", 0, 19))>;
+def CSR_64_AllRegs_AIX_Dflt_VSX : CalleeSavedRegs<(add CSR_64_AllRegs_AIX_Dflt_Altivec,
+                                         (sequence "VSL%u", 0, 31))>;
 
----------------
Thanks for catching this. 

I don't quite understand why it's ok to set it to `31` here even though it seems to work. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D133547



More information about the llvm-commits mailing list