[PATCH] D49935: [AArch64] Support the .inst directive for MachO and COFF targets

[Martin Storsjö via Phabricator via llvm-commits]

mstorsjo added inline comments.


================
Comment at: lib/Target/AArch64/MCTargetDesc/AArch64TargetStreamer.cpp:47
+  // symbol, and swap the endianness on big-endian systems (instructions are
+  // always little-endian).
+  for (unsigned I = 0; I < 4; ++I) {
----------------
efriedma wrote:
> This comment doesn't make any sense: we don't use this codepath on targets with mapping symbols.
Oh, indeed, I had meant to keep it for the endianness part.


Repository:
  rL LLVM

https://reviews.llvm.org/D49935