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

Matthias Braun via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 29 11:20:16 PDT 2016


> 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


More information about the llvm-commits mailing list