[PATCH] D83289: [BPF] Emit unknown types as byte arrays

Martin KaFai Lau via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 9 11:16:21 PDT 2020


iamkafai added a comment.

I don't have strong opinion on whether using int for float/double or byte array or completely skip emitting BTF for those members.  I would slightly prefer the later (skip emitting BTF) for now until we figure out a real use case that the bpf prog needs to read them.  I think the bpf program is not supposed to access the float/double anyway, right?.

I would prefer a warning.  especially for >64k members which should not happen, right?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83289





More information about the llvm-commits mailing list