<div dir="ltr">Thanks for the update!<br><div>I agree that the probe based part can be moved to the base later. Once all the lineno based code is in place, moving code should be straightforward.<br></div><div><br></div><div>-Rong</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Feb 16, 2021 at 10:51 AM Hongtao Yu 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">hoy accepted this revision.<br>
hoy added a comment.<br>
<br>
Sorry for the late response. We've been thinking about the integration with CSSPGO. May need to move more stuff (like `getProbeWeight`) into the base class for MIR support.<br>
<br>
Current changes look good to me. Thanks.<br>
<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D96455/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D96455/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D96455" rel="noreferrer" target="_blank">https://reviews.llvm.org/D96455</a><br>
<br>
</blockquote></div>