[PATCH] D19401: MachineScheduler: Fully compare top/bottom candidates

Matthias Braun via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 20 16:50:15 PDT 2016


MatzeB added a comment.

I would like to move this patch forward. I analyze some of the AMDGPU regression and they are accidentally cause by shortcomings in the MachineScheduler ReadyCycle computation which calculates suboptimal values when an instruction requires 0 cycles. Fixing this properly is for another day, but the obvious fix for AMDGPU is not defining COPY as 0 cycles (which most likely was not intentional anyway). Tom or Matt could you take a look at: http://reviews.llvm.org/D21540?


Repository:
  rL LLVM

http://reviews.llvm.org/D19401





More information about the llvm-commits mailing list