[PATCH] D72225: Align branches within 32-Byte boundary(Prefix padding)

Kan Shengchen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 10 00:00:54 PST 2020


skan marked an inline comment as done.
skan added inline comments.


================
Comment at: llvm/test/MC/X86/align-branch-64-2d.s:1
+# Check only indirect jumps and calls are aligned with option --x86-align-branch-boundary=32 --x86-align-branch=indirect+call --x86-align-branch-prefix-size=4
+# RUN: llvm-mc -filetype=obj -triple x86_64-unknown-unknown --x86-align-branch-boundary=32 --x86-align-branch=indirect+call --x86-align-branch-prefix-size=4 %p/Inputs/align-branch-64-2.s  | llvm-objdump -d  - | FileCheck %s
----------------
MaskRay wrote:
> When writing tests, make sure llvm-mc and GNU as emit jmp of the same length. There are differences (D72197).
According to my understanding, you are referring to the reallocation of call? I will update the tests when D72197 is landed.


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

https://reviews.llvm.org/D72225





More information about the llvm-commits mailing list