[llvm] [MachineScheduler][ScheduleDAG] Add ability to bias scheduling by longest paths (PR #93223)
Michael Maitland via llvm-commits
llvm-commits at lists.llvm.org
Tue May 28 14:14:15 PDT 2024
michaelmaitland wrote:
> Then there's a bug in the patch, the compiler output should always be deterministic
It turns out that the non-determinism only existed in our downstream and was related to a different issue. That separate issue led me to believe that this patch was helping to improve scheduling. I suppose this patch helped that non-determinism show itself. After resolving the non-determinism in our downstream, the gains we saw "due to this patch" went away. As a result, I am closing this PR since it does not seem to help much.
https://github.com/llvm/llvm-project/pull/93223
More information about the llvm-commits
mailing list