[PATCH] D49935: [AArch64] Support the .inst directive for MachO and COFF targets
Martin Storsjö via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jul 27 21:41:31 PDT 2018
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
More information about the llvm-commits
mailing list