[PATCH] D99053: [RISCV] Add scheduler classes to Zfh instructions.

[Hsiangkai Wang via Phabricator via llvm-commits]

HsiangKai added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVInstrInfoZfh.td:89
 def FMSUB_H  : FPFMAH_rrr_frm<OPC_MSUB, "fmsub.h">,
-               Sched<[]>;
+               Sched<[WriteFMulSub16, ReadFMulSub16, ReadFMulSub16, ReadFMulSub16]>;
 def          : FPFMAHDynFrmAlias<FMSUB_H, "fmsub.h">;
----------------
craig.topper wrote:
> Does anyone know why we have different classes for FMADD and FMSUB? It doesn't seem like they are likely to be different.
I agree. We should combine them into something like WriteFMulAddSub.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99053