[PATCH] D124092: CodeGen: Remove MaxBytesForAlignment from MachineBasicBlock

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 1 07:05:14 PDT 2022


arsenm added a comment.

In D124092#3550117 <https://reviews.llvm.org/D124092#3550117>, @dmgreen wrote:

> I'm not sure I understand. Why do you think that MaxBytesForAlignment isn't a per-basic-block parameter? Any aligned block can have a MaxBytesForAlignment, and they could all be different from one block to the next, just as alignments can change.

But that isn't actually used. This is only ever sourced from a target constant value. This is an unused feature essentially


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

https://reviews.llvm.org/D124092



More information about the llvm-commits mailing list