[PATCH] D78387: [AIX][XCOFF] add symbol priority for the llvm-objdump -D -symbol-description

Digger via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 22 05:56:00 PDT 2020


DiggerLin marked 2 inline comments as done.
DiggerLin added inline comments.


================
Comment at: llvm/include/llvm/MC/MCDisassembler/MCDisassembler.h:60
+             std::tie(P2.Addr, P2.XCOFFSymInfo, P2.Name);
+    } else
       return std::tie(P1.Addr, P1.Name, P1.Type) <
----------------
I will delete the "else" in the next patch update


================
Comment at: llvm/lib/MC/MCDisassembler/MCDisassembler.cpp:79
+
+/// The function is for symbol sorting when symbols have the same address. if
+/// the symbols of the same section are sorted in ascending order. The
----------------
I will delete the "if" in next patch update.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78387





More information about the llvm-commits mailing list