<div dir="ltr"><div class="gmail_default" style="font-family:monospace;font-size:small;color:#000000">One time exception sounds fine to me.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jun 21, 2021 at 4:53 PM Fangrui Song 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">MaskRay added a comment.<br>
<br>
Not keeping version compatibility for the raw profile format would actually make my life easier.<br>
But I was CCed on the Linux kernel news <a href="https://www.phoronix.com/scan.php?page=news_item&px=Clang-PGO-For-Linux-Next" rel="noreferrer" target="_blank">https://www.phoronix.com/scan.php?page=news_item&px=Clang-PGO-For-Linux-Next</a> so I paid additional care in this patch.<br>
<br>
As we can see the compatibility code is actually minimal: just `if (relativeCounterPtr())` and the associated function.<br>
There comments are longer than the code.<br>
<br>
How about we making a one-time exception as a courtesy to the initial check-in of the `CONFIG_PGO_CLANG` feature in the Linux kernel?<br>
There can be a bunch of users testing the PGO functionality.<br>
(Well, you may argue that why I couldn't postpone this patch for some extended time, 1 month? I had this in mind for a while (when I changed XRay and learned a bit about PGO last year I had a plan) but I would probably have postponed this work further if I had not seen that some folks are proposing the MIP profile format.)<br>
<br>
The summary can be clarified that this is a one-time exception.<br>
I can set up a calendar reminder for myself to drop the compatibility code in llvm-project after some time (@nickdesaulniers @kees may decide on this).<br>
<br>
<br>
Repository:<br>
  rG LLVM Github Monorepo<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D104556/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D104556/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D104556" rel="noreferrer" target="_blank">https://reviews.llvm.org/D104556</a><br>
<br>
</blockquote></div>