[PATCH] D76739: [llvm-objdump] Replace array_pod_sort with llvm::stable_sort

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 26 09:12:25 PDT 2020


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


================
Comment at: llvm/tools/llvm-objdump/llvm-objdump.cpp:1207
+  // cause more than one section at the same address. Use a stable sort to
+  // stabalize the output.
   std::vector<std::pair<uint64_t, SectionRef>> SectionAddresses;
----------------
jhenderson wrote:
> Here and below: stabalize -> stabilize.
Thanks! Fixed by 579a7a1938403c272706e91fe6c9367ff682eea1


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76739





More information about the llvm-commits mailing list