[PATCH] D55756: [DebugInfo] Make AsmPrinter struct HandlerInfo and Handlers protected

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


yonghong-song added a comment.

@echristo @aprantl @dblaikie
This patch and two other patches (D55755 <https://reviews.llvm.org/D55755>, 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/D55756/new/

https://reviews.llvm.org/D55756





More information about the llvm-commits mailing list