<div dir="ltr">I don' t have issue with that patch, but you need rsmith@ to take a closer look at it.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Feb 28, 2019 at 9:18 AM Jonas Hahnfeld via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hahnfeld added a comment.<br>
<br>
In D58787#1413869 <<a href="https://reviews.llvm.org/D58787#1413869" rel="noreferrer" target="_blank">https://reviews.llvm.org/D58787#1413869</a>>, @davidxl wrote:<br>
<br>
> 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.<br>
<br>
<br>
Yeah I know, the other approach is making the test resilient against such changes in ordering: D58385 <<a href="https://reviews.llvm.org/D58385" rel="noreferrer" target="_blank">https://reviews.llvm.org/D58385</a>><br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D58787/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D58787/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D58787" rel="noreferrer" target="_blank">https://reviews.llvm.org/D58787</a><br>
<br>
<br>
<br>
</blockquote></div>