[PATCH] D55755: [DebugInfo] Move several private headers to include directory

Yonghong Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 16 22:02:04 PST 2018


yonghong-song added a comment.

@echristo @aprantl @dblaikie 
This patch and two other patches (D55756 <https://reviews.llvm.org/D55756>, D55752 <https://reviews.llvm.org/D55752>) implemented a different mechanism
to generate BTF than patch D53736 <https://reviews.llvm.org/D53736>. D53736 <https://reviews.llvm.org/D53736> has its most implementation under
lib/CodeGen/AsmPrinter. The new mechanism put most implementation under BTF target.

  (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

It will be good if you can help review all related patches and provide your opinions soon.
Thanks! Real appreciated.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D55755





More information about the llvm-commits mailing list