[PATCH] D130902: [llvm-objdump,ARM] Fix big-endian AArch32 disassembly.

Fri Aug 5 08:58:37 PDT 2022

DavidSpickett added a comment.

LGTM if @MaskRay is cool with it.

I remembered that this patch https://reviews.llvm.org/D48811 was posted ages ago and never went anywhere. Do you have an opinion on that? It makes objdump disassemble as big endian if there is `eb` in the target name. From what I read here I don't think it's needed because the important thing is the format in the object file. Is that correct?

================
Comment at: llvm/lib/Target/ARM/ARM.td:733
+//===----------------------------------------------------------------------===//
+// Endianness of instruction encodings in memory.
+//
----------------
`// Endianness of instruction encodings`

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130902