Fix the MachineScheduler's logic for updating ready times for in-order.
authorAndrew Trick <atrick@apple.com>
Sat, 7 Jun 2014 01:48:43 +0000 (01:48 +0000)
committerAndrew Trick <atrick@apple.com>
Sat, 7 Jun 2014 01:48:43 +0000 (01:48 +0000)
commit7f1ebbeb8fdcae821ec5ae32d3bcb6c428016663
tree0f061f0103490173db93e676f8ff9f988174b6a3
parent86f673a0055120a3dfae56b96a0dde803ea0348c
Fix the MachineScheduler's logic for updating ready times for in-order.

Now the scheduler updates a node's ready time as soon as it is
scheduled, before releasing dependent nodes. There was a reason I
didn't do this initially but it no longer applies.

A53 is in-order and was running into an issue where nodes where added
to the readyQ too early. That's now fixed.

This also makes it easier for custom scheduling strategies to build
heuristics based on the actual cycles that the node was scheduled at.

The only impact on OOO (sandybridge/cyclone) is that ready times will
be slightly more accurate. I didn't measure any significant regressions.

llvm-svn: 210390
llvm/include/llvm/CodeGen/MachineScheduler.h
llvm/lib/CodeGen/MachineScheduler.cpp