[PATCH] D68822: [BPF] Support external globals

Andrii Nakryiko via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 21 20:13:37 PDT 2019


anakryiko added a comment.

if there is no type information available for extern variables, there is little benefit in having them as DATASEC+VAR. No size, no type. We can get the same information from ELF itself without having to teach verifier and libbpf how to handle these new types. Let's drop this for now? The only potential benefit is having a section name associated with extern, but we can either add that later, or even add it in a way that will require no verifier changes - BTF.ext.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D68822





More information about the llvm-commits mailing list