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

[James Henderson via Phabricator via llvm-commits]

jhenderson added a comment.

Not being experienced with AARCH64, what does the endianness impact? Does it affect the encoding of instructions, because if so, I'm surprised that the .plt section disassembles fine in this case.



================
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
----------------
Any particular reason you've removed the new line? The line is getting a little long here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96211