[PATCH] D110561: [AArch64] Alter arith-cbz-fusion to fuse between pairs register instructions

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 27 11:56:33 PDT 2021


MatzeB added a comment.

> It seems to be between an SUnit and its Preds. But there can be many dependencies between a SUnit and ExitSU (the CBZ), more than just where register dependencies exist.

I don't think most shouldScheduleAdjacent implementations make sense otherwise. This line looks like it rejects the other dependency types: https://github.com/llvm/llvm-project/blob/06e2a0684e52b6cd2e4af3e19964bb5953573465/llvm/lib/CodeGen/MacroFusion.cpp#L180


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D110561/new/

https://reviews.llvm.org/D110561



More information about the llvm-commits mailing list