[PATCH] D76166: [MC] Handle layout change due to relaxation of another section

Jian Cai via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 13 17:17:41 PDT 2020


jcai19 added a comment.

Thanks! I actually had almost identical implementation at first :), but then I realized I needed to avoid "stats::FragmentLayouts" to be increased in MCAsmLayout::layoutFragment when adjusting the offset of valid fragments and changed my implementation to https://reviews.llvm.org/D76114?id=250118. Also, one thing I am not sure is if marking fragments invalid will always trigger relaxation if needed in the following iteration of the while loop?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76166





More information about the llvm-commits mailing list