[PATCH] D98002: [RISCV] Add scheduling resources for V

[Fraser Cormack via Phabricator via llvm-commits]

frasercrmck added a comment.

LGTM too, thanks @evandro! Just one last question I was wondering about regarding reductions.



================
Comment at: llvm/lib/Target/RISCV/RISCVInstrInfoV.td:593
+  def _VS : VALUVV<funct6, OPMVV, opcodestr # ".vs">,
+            Sched<[WriteVIRedV, ReadVIRedV, ReadVIRedV, ReadVMask]>;
+}
----------------
I dunno if the first source of a reduction is likely to benefit from a different scheduling type than the second, since it's only reading the first element?


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

https://reviews.llvm.org/D98002