[PATCH] D70157: Align branches within 32-Byte boundary(NOP padding)

Kan Shengchen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 19 23:03:37 PST 2019


skan marked an inline comment as done.
skan added inline comments.


================
Comment at: llvm/lib/MC/MCAssembler.cpp:1003
+  // exists) also marks the end of the branch.
+  for (auto i = 0U, N = BF.isFused() ? 2U : 1U;
+       i != N && !isa<MCBoundaryAlignFragment>(F); ++i, F = F->getNextNode()) {
----------------
MaskRay wrote:
> unsigned -> int
Why use `int` here?


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

https://reviews.llvm.org/D70157





More information about the llvm-commits mailing list