[PATCH] D99040: [RISCV] Add scheduler classes for the Zba and Zbb extensions.

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVSchedRocket.td:177
 
+let Unsupported = true in {
+def : WriteRes<WriteRotateImm, []>;
----------------
evandro wrote:
> Perhaps consider placing the resources for the B instructions when they are not supported in a separate file for convenient inclusion in targets which do not support it.
I'm worried that starts becoming unmanageable. You would need a file per sub extension which will quickly pollute the directory.

Maybe we need a multiclass to instantiate instead?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99040