[PATCH] D75404: [X86] Not track size of the boudaryalign fragment during the layout

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Mar 1 17:08:31 PST 2020


MaskRay added inline comments.


================
Comment at: llvm/lib/MC/MCAssembler.cpp:366
+    // MCBoundaryAlignFragment(if exists) also marks the end of the branch.
+    for (auto I = 0, N = BF.isFused() ? 2 : 1;
+         I != N && !isa<MCBoundaryAlignFragment>(F);
----------------
`auto` -> `int`


================
Comment at: llvm/lib/MC/MCAssembler.cpp:368
+         I != N && !isa<MCBoundaryAlignFragment>(F);
+         ++I, F = F->getNextNode()) {
+      AlignedSize += computeFragmentSize(Layout, *F);
----------------
Delete excess parentheses.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75404





More information about the llvm-commits mailing list