[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