jrbyrnes accepted this revision.
jrbyrnes added a comment.
This revision is now accepted and ready to land.
LGTM
================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.cpp:427
DAG->addMutation(createStoreClusterDAGMutation(DAG->TII, DAG->TRI));
DAG->addMutation(createIGroupLPDAGMutation());
DAG->addMutation(createAMDGPUMacroFusionDAGMutation());
----------------
kerbowa wrote:
> jrbyrnes wrote:
> > I think you can remove this as well since you're doing it from within the
> > scheduler.
> It's not added in the scheduler for plain SCHED_BARRIER.
Oh okay -- I see
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D132079/new/
https://reviews.llvm.org/D132079
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
