<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jun 18, 2020 at 3:22 PM David Li 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">davidxl added a comment.<br>
<br>
ok. Why atomic update just for first entry? are they particularly more contended than other counters?<br></blockquote><div>I don't think they have more contention than other counters. </div><div>They are more important than other counters as the entry count is the base for profile count in BFI. </div><div>Previously I have done some work to verify the BFI using profile count. I think that helped (but it slows down the instrumentation a lot)</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D82123/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D82123/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D82123" rel="noreferrer" target="_blank">https://reviews.llvm.org/D82123</a><br>
<br>
<br>
<br>
</blockquote></div></div>