[llvm] MachineScheduler: Improve instruction clustering (PR #137784)

Jay Foad via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 29 04:55:47 PDT 2025


jayfoad wrote:

> The existing implemention has some buggy behavior: The scheduler does not reset the pointer to next cluster candidate. For example, we want to cluster A and B, but after picking A, we might pick node C. In theory, we should reset the next cluster candiate here, because we have decided not to cluster A and B during scheduling. Later picking B because of Cluster seems not logical.

Could you fix that bug first in a separate PR? Then this PR should have fewer test check changes.

https://github.com/llvm/llvm-project/pull/137784


More information about the llvm-commits mailing list