[PATCH] D58787: [ProfileData] Sort ProfilingData by hash

Jonas Hahnfeld via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 28 09:18:00 PST 2019


Hahnfeld added a comment.

In D58787#1413869 <https://reviews.llvm.org/D58787#1413869>, @davidxl wrote:

> The secondary map 'ProfileData' is needed occasionally (to use cfg hash).  doing sorting with std::map is not needed strictly speaking, but it has the nice property of having fixed order.


Yeah I know, the other approach is making the test resilient against such changes in ordering: D58385 <https://reviews.llvm.org/D58385>


Repository:
  rL LLVM

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

https://reviews.llvm.org/D58787





More information about the llvm-commits mailing list