[PATCH] D74349: [PowerPC][AIX] Spill and restore the non-volatile condition register bits.

Mon Feb 24 05:01:40 PST 2020

cebowleratibm added inline comments.

================
Comment at: llvm/lib/Target/PowerPC/PPCFrameLowering.cpp:929
+      MachineInstrBuilder MIB =
+          BuildMI(MBB, MBBI, dl, TII.get(PPC::MFOCRF8), TempReg);
+      MIB.addReg(MustSaveCRs[0], RegState::Kill);
----------------
Xiangling_L wrote:
> minor issue: inconsistency between how to get instruction `TII.get(PPC::MFOCRF8)`  and `MoveFromCondRegInst`?
I think the current code is the preferred form.  I had asked for the IsPPC64 assertion above to clarify to the reader that it's ok to hard code MFOCRF8.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74349