[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