[PATCH] D72738: [BranchAlign] Add master --x86-branches-within-32B-boundaries flag

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added inline comments.


================
Comment at: llvm/test/MC/X86/align-branch-64-1a.s:7
+# RUN: llvm-mc -filetype=obj -triple x86_64-unknown-unknown --x86-branches-within-32B-boundaries %p/Inputs/align-branch-64-1.s | llvm-objdump -d  - > %t1
+# RUN: FileCheck --input-file=%t1 %s
+
----------------
MaskRay wrote:
> Compare the llvm-mc output with the previous one.
> 
> `cmp %t1 %t2` returns 0 => identical output.
```
llvm-mc -filetype=obj -triple x86_64 --x86-align-branch-boundary=32 --x86-align-branch=fused+jcc+jmp %p/Inputs/align-branch-64-1.s -o %t1
llvm-objdump -d %t1 | FileCheck %s

llvm-mc ... -o %t2
cmp %t1 %t2
```

Compare the object files, not the llvm-objdump output.


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

https://reviews.llvm.org/D72738