[llvm] [RISCV] Add SiFive X390 scheduling model (PR #143938)
Pengcheng Wang via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 17 20:08:29 PDT 2025
wangpc-pp wrote:
> > LGTM. I still feel uncomfortable about the complexity (too many conditions, and I personally prefer straightforward definitions) but I think we can improve it progressively.
>
> Yeah I agree the complexity is a bit daunting. I feel like fundamentally the scheduling model infrastructure should be more ergonomic on creating hierarchical scheduling models. In fact, many many moons ago on the inception of the current instr scheduling model framework, I think people _were_ thinking about making a way to have a "base" / parent scheduling model (there is a comment left somewhere in the TableGen code that shows this) but obviously it went nowhere.
I had the same plan but I didn't have much time to give it a try. Please feel free to tag me when you are trying to fix these scheduling model issues. :-)
https://github.com/llvm/llvm-project/pull/143938
More information about the llvm-commits
mailing list