[PATCH] D76285: [X86][MC] Fix the bug for prefix padding support

Thu Mar 19 22:27:10 PDT 2020

LuoYuanke added inline comments.

================
Comment at: llvm/test/MC/X86/align-branch-64-pad-max-prefix.s:10
+  # We should not increase the length of this jmp to reduce the bytes of
+  # following nops, doing so would make the jmp misaligned.
+# CHECK:      18:          jmp
----------------
skan wrote:
> reames wrote:
> > This case should already be handled by skipping instructions past the branch align.
> No, it's not handled.  Your can run this test case without this patch to prove.
Do you mean when jmp size is expanded, it cross the align boundary? That seems a problem. The jmp size should be frozen after nop padding.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76285