<div dir="ltr">This is really great. I've gotten the review in the next patches.<div><br></div><div>Thanks!</div></div><br><div class="gmail_quote"><div dir="ltr">On Sun, Dec 16, 2018 at 9:56 PM Yonghong Song via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">yonghong-song added a comment.<br>
<br>
@echristo @aprantl @dblaikie This patch has stuck in this state for more than one month and did not make progress.<br>
I have suggested a possible way to move some header files to include directory and then we will be<br>
able to move BTFDebug.{cpp,h} under BPF directory. I have implemented this approach with the<br>
following three patches for easy to see pro and cons of two approaches:<br>
<br>
  (1) <a href="https://reviews.llvm.org/D55755" rel="noreferrer" target="_blank">https://reviews.llvm.org/D55755</a>, moving header files<br>
  (2) <a href="https://reviews.llvm.org/D55756" rel="noreferrer" target="_blank">https://reviews.llvm.org/D55756</a>, changes some fields from private to protected<br>
  (3) <a href="https://reviews.llvm.org/D55752" rel="noreferrer" target="_blank">https://reviews.llvm.org/D55752</a>, changes only under target BPF<br>
<br>
The new patch series tried to minimize the changes outside the BPF target.<br>
I will really appreciate if you can timely review this patch and related other "competing" patches.<br>
Thanks!<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D53736/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D53736/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D53736" rel="noreferrer" target="_blank">https://reviews.llvm.org/D53736</a><br>
<br>
<br>
<br>
</blockquote></div>