[PATCH] D39214: [MachineModuleInfoImpls] Replace qsort with std::stable_sort

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 23 16:33:34 PDT 2017


efriedma added a comment.

Using stable_sort doesn't solve anything here: the input is constructed by iterating over a DenseMap whose key is a pointer.

(If you just want to clean this up without actually changing the algorithm, there's a three-operand overload of array_pod_sort.)


Repository:
  rL LLVM

https://reviews.llvm.org/D39214





More information about the llvm-commits mailing list