[PATCH] D96211: [llvm-objdump] Support PLT decoding for aarch64_be

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 8 00:44:36 PST 2021


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


================
Comment at: llvm/test/tools/llvm-objdump/ELF/AArch64/plt.test:9
 # RUN: yaml2obj %s -o %t.aarch64
-# RUN: llvm-objdump -d --mattr=+bti %t.aarch64 | \
-# RUN:   FileCheck --check-prefix=CHECK-BTI %s
+# RUN: llvm-objdump -d --mattr=+bti %t.aarch64 | FileCheck --check-prefix=CHECK-BTI %s
+# RUN: yaml2obj -DENDIAN=MSB %s -o %t.aarch64_be
----------------
jhenderson wrote:
> Any particular reason you've removed the new line? The line is getting a little long here.
86 bytes... which is fine


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96211



More information about the llvm-commits mailing list