[llvm] [MachineScheduler][AMDGPU] Allow scheduling of single-MI regions (PR #128739)
Lucas Ramirez via llvm-commits
llvm-commits at lists.llvm.org
Wed Feb 26 09:51:49 PST 2025
lucas-rami wrote:
> Testcase where this makes a difference?
I spent some time trying to come up with one but didn't manage to. This change is motivated by ongoing pre-RA rematerialization work on the AMDGPU backend (see #125885) which requires that we are able to track regions with a single MI. The current implementation follows a different logic to identify rematerializable instructions, and I can't manage to make this fail/succeed depending on whether single-MI regions are filtered out/in.
> Am I looking at the right diff here? The only code change shown here is the removal of I == std::prev(RegionEnd).
A few unit tests across various targets were also updated manually.
https://github.com/llvm/llvm-project/pull/128739
More information about the llvm-commits
mailing list