[PATCH] D53736: [BTF] Add BTF DebugInfo

Sun Dec 16 21:56:19 PST 2018

yonghong-song added a comment.

@echristo @aprantl @dblaikie This patch has stuck in this state for more than one month and did not make progress.
I have suggested a possible way to move some header files to include directory and then we will be
able to move BTFDebug.{cpp,h} under BPF directory. I have implemented this approach with the
following three patches for easy to see pro and cons of two approaches:

  (1) https://reviews.llvm.org/D55755, moving header files
  (2) https://reviews.llvm.org/D55756, changes some fields from private to protected
  (3) https://reviews.llvm.org/D55752, changes only under target BPF

The new patch series tried to minimize the changes outside the BPF target.
I will really appreciate if you can timely review this patch and related other "competing" patches.
Thanks!

Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D53736/new/

https://reviews.llvm.org/D53736