ZarkoCA added inline comments.
================ Comment at: llvm/lib/Target/PowerPC/PPCRegisterInfo.cpp:235 + return TM.isPPC64() + ? (Subtarget.hasAltivec() ? CSR_64_AllRegs_Altivec_RegMask + : CSR_PPC64_RegMask) ---------------- sfertile wrote: > `CSR_64_AllRegs_Altivec_RegMask` should be `CSR_PPC64_Altivec_RegMask`. FWIW > I don't think this is testable without D86476. If that's the case, then it > should go in that patch, not this patch. Are you suggesting that I also leave the error in if I were to move this change to D84676? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88676/new/ https://reviews.llvm.org/D88676 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
