[PATCH] D150671: RISCV/InstrInfo: model register pressure for MICombiner

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 6 08:43:38 PDT 2023


craig.topper added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVInstrInfo.cpp:1412
+
+bool RISCVInstrInfo::shouldReduceRegisterPressure(
+    const MachineBasicBlock *MBB, const RegisterClassInfo *RegClassInfo) const {
----------------
Does this get called for register classes other than GPR?

If the GPR pressure is already over the limit, does that mean we return true even though the register class in question might be floating point?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150671



More information about the llvm-commits mailing list