[PATCH] D76114: [MC] Recalculate fragment offsets after relaxation

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 13 16:45:11 PDT 2020


reames added a comment.

In D76114#1922553 <https://reviews.llvm.org/D76114#1922553>, @jcai19 wrote:

> > This does not appear to relate to the recent alignment padding work.  Test continues to fail with -x86-pad-for-align=0 -x86-pad-for-branch-align=0
>
> I am not sure I followed here but I ran the test case as follows and it still passed.
>
> llvm-mc -filetype=obj -triple=i386 relax-offset.s -o /dev/null -x86-pad-for-align=0 -x86-pad-for-branch-align=0


Er, I assume you mean "passed on the build with your change" correct?

I specifically meant "continued to fail on top of tree with those padding flags disabled (set to 0)".


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76114





More information about the llvm-commits mailing list