[PATCH] D53261: [BPF] Add BTF generation for BPF target

Adrian Prantl via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 16 13:59:01 PDT 2018


aprantl added inline comments.


================
Comment at: include/llvm/MC/MCBTFContext.h:167
+/// The .BTF.ext section header definition
+struct BTFExtHeader {
+  uint16_t Magic;
----------------
yonghong-song wrote:
> aprantl wrote:
> > do you need any kind of packing attribute here for this to work on all platforms?
> The structure here is designed to be no holes based on C standard. Do you know which platform could require packing?
No, I'm no expert on the C record layout; that's why I was asking :-)


Repository:
  rL LLVM

https://reviews.llvm.org/D53261





More information about the llvm-commits mailing list