[llvm] [MCA][ResourceManager] Fix a bug in the instruction issue logic. (PR #108386)
Aiden Grossman via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 12 10:24:27 PDT 2024
https://github.com/boomanaiden154 approved this pull request.
LGTM. Thanks for the fix!
Algorithm seems reasonable enough to me. It seems like it would handle all cases that would possibly scheduable, and the worst-case quadratic complexity shouldn't be too bad. I'd imagine even instructions with a quite a few resource groups wouldn't perform terribly, but should be exceptionally rare.
https://github.com/llvm/llvm-project/pull/108386
More information about the llvm-commits
mailing list