[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]

craig.topper added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:15154
+
+  if (LocVT == MVT::f16 && Subtarget.hasStdExtZvfh()) {
     static const MCPhysReg FPR16List[] = {
----------------
I think you meant Zfh? But even then you probably mean Zfh || Zfhmin?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D155502