[PATCH] D155502: [RISCV] Do not use F registers if zvfh/f/d is not specified in the architecture

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 17 12:05:38 PDT 2023


craig.topper added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:15191
 
   if (LocVT == MVT::i32 || LocVT == MVT::f32) {
     unsigned Offset4 = State.AllocateStack(4, Align(4));
----------------
If we skip the FPRs above, should we be going through GPR before we hit this stack code?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D155502



More information about the llvm-commits mailing list