[PATCH] D59441: [BPF] Add BTF Var and DataSec Support

[Alexei Starovoitov via Phabricator via llvm-commits]

ast accepted this revision.
ast added inline comments.
This revision is now accepted and ready to land.
Herald added a subscriber: ormris.


================
Comment at: lib/Target/BPF/BTF.h:174
+  VAR_GLOBAL_ALLOCATED = 1, ///< Linkage: ExternalLinkage
+  VAR_GLOBAL_TENTATIVE = 2, ///< Linkage: CommonLinkage
+  VAR_GLOBAL_EXTERNAL = 3,  ///< Linkage: ExternalLinkage
----------------
this two are unused. Reserved for future ?


================
Comment at: lib/Target/BPF/BTFDebug.cpp:326
+  BTFType.Info = Kind << 24;
+  BTFType.Size = 0;
+}
----------------
too bad that compiler cannot determine it at the point of btf emission.
i think it's ok to ask libbpf to fix it up.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D59441