[PATCH] D65992: [MachinePipeliner] Avoid indeterminate order in FuncUnitSorter

James Molloy via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 9 01:21:57 PDT 2019


jmolloy accepted this revision.
jmolloy added a comment.
This revision is now accepted and ready to land.

LGTM. Note for other reviewers; the canonical way to avoid this in LLVM is to use a stable_sort. The reason we can't do this is because this functor is used as the comparison for a std::priority_queue (heap) rather than a plain sort.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65992





More information about the llvm-commits mailing list