Most Arm disassemblers, including GNU objdump and Arm's own fromelf,
emit an instruction's raw encoding as a 32-bit words or (for Thumb)
one or two 16-bit halfwords, in logical order rather than according to
their storage endianness. This is generally easier to read: it matches
the encoding diagrams in the architecture spec, it matches the value
you'd write in a .inst directive, and it means that fields within
the instruction encoding that span more than one byte (such as branch
offsets or SVC immediates) can be read directly in the encoding
without having to mentally reverse the bytes.
llvm-objdump already has a system of PrettyPrinter subclasses which
makes it easy for a target to drop in its own preferred formatting.
This patch adds pretty-printers for all the Arm targets, so that
llvm-objdump will display Arm instruction encodings in their preferred
layout instead of little-endian and bytewise.
You could write this as:
size_t insn_bytes = STI.checkFeatures("+thumb-mode") ? 2 : 4; for (; Pos + insn_bytes <= End; Pos += insn_bytes) OS << ' ' << format_hex_no_prefix( llvm::support::endian::read<uint16_t>( Bytes.data() + Pos, llvm::support::little), insn_bytes*2); }