[PATCH] D24855: MachineScheduler: Enable macro fusion in post-RA scheduler

[Andrew Trick via llvm-commits]

> On Sep 29, 2016, at 11:20 AM, Matthias Braun <matze at braunis.de> wrote:
> 
> 
>> On Sep 28, 2016, at 10:01 PM, Andrew Trick via llvm-commits <llvm-commits at lists.llvm.org> wrote:
>> 
>> atrick added a comment.
>> 
>> Marking a DAG node as "dead" for the purpose of scheduling should be an easy thing to do, relative to supporting instruction bundles. But adding the extra DAG edges is also a fine solution, just not quite as direct.
> 
> Marking a DAG as dead is still a more complex patch than this which is a strict improvement over the status quo and "contained" within the MacroFusionDAGMutation. So is this good to go?
> 
> - Matthias

The review site seems down… I glanced at your attached patch. It looks like you’re traversing every edge in the DAG. Why don’t you just revisit the DAG leaves and immediate predecessors of the exitSU?

-Andy