[PATCH] D75404: [X86] Not track size of the boudaryalign fragment during the layout
Kan Shengchen via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Mar 1 17:47:21 PST 2020
skan marked an inline comment as done.
skan added inline comments.
================
Comment at: llvm/lib/Target/X86/MCTargetDesc/X86AsmBackend.cpp:445
// If the branch is emitted into a MCRelaxableFragment, we can determine the
- // size of the branch easily in MCAssembler::relaxBoundaryAlign. When the
+ // size of the branch easily in during the process of layout. When the
// branch is fused, the fused branch(macro fusion pair) must be emitted into
----------------
MaskRay wrote:
> easily during the layout process.
oh..I missed this comment. Currently there are many redundant boundary align fragments inserted, I will open another revison to fix that and of course the code change will cover this comment.
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