[PATCH] D125008: [llvm-objdump] Print Mnemonic Histogram

Thorsten via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 5 09:00:41 PDT 2022


tschuett added inline comments.


================
Comment at: llvm/tools/llvm-objdump/llvm-objdump.cpp:2171
+
+  std::sort(List.begin(), List.end(), [] (std::pair<std::string, unsigned> &L,
+                                          std::pair<std::string, unsigned> &R) {
----------------
DavidSpickett wrote:
> Probably makes no difference but why not `const std::pair<std::string, unsigned> &`.
Maybe std::stable_sort to make it deterministic?


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

https://reviews.llvm.org/D125008



More information about the llvm-commits mailing list