[llvm] [RISCV] Split sched classes for vrgather.vv and vrgatherei16.vv (PR #92768)
Craig Topper via llvm-commits
llvm-commits at lists.llvm.org
Mon May 20 12:52:39 PDT 2024
================
@@ -1677,8 +1677,9 @@ let Predicates = [HasVInstructions] in {
let Constraints = "@earlyclobber $vd", RVVConstraint = Vrgather in {
defm VRGATHER_V : VGTR_IV_V_X_I<"vrgather", 0b001100>;
def VRGATHEREI16_VV : VALUVV<0b001110, OPIVV, "vrgatherei16.vv">,
- SchedBinaryMC<"WriteVRGatherVV", "ReadVRGatherVV_data",
- "ReadVRGatherVV_index">;
+ SchedBinaryMC<"WriteVRGatherVVEEW",
----------------
topperc wrote:
> It is meant to convey that it is used for vrgatherei16.vv and not vrgather.vv. Would it be better if we did WriteVRGatherEI16VV
I think so
> I put EEW at the end because of VPseudoVGTR_VV_EEW naming
Looks like that takes `eew` as a template parameter. So we should use that parameter to form the scheduler class name. Or maybe we should change the name and drop that parameter?
https://github.com/llvm/llvm-project/pull/92768
More information about the llvm-commits
mailing list