<div dir="ltr">Machine level profile loader will use the SampleCoverage Tracker. This class will be kept in the templated header. Machine level profile loader and IR level profile loader will be using the exactly the same class (not templatelized). </div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Feb 1, 2021 at 4:01 PM Wei Mi 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">wmi added a comment.<br>
<br>
> This patch detaches SampleProfileLoader from SampleCoverageTracker. This would keep SampleCoverageTracker as a class.<br>
<br>
It means you don't want SampleCoverageTracker to be a template. Will the FS profile annotation in machine code level also need some coverage tracker?<br>
<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D95823/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D95823/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D95823" rel="noreferrer" target="_blank">https://reviews.llvm.org/D95823</a><br>
<br>
</blockquote></div>