[PATCH] D150079: [BPF][DebugInfo] Show CO-RE relocations in llvm-objdump

Thu Aug 24 16:07:57 PDT 2023

eddyz87 marked an inline comment as done.
eddyz87 added inline comments.

================
Comment at: llvm/lib/DebugInfo/BTF/BTFParser.cpp:332

-Error BTFParser::parse(const ObjectFile &Obj) {
+Error BTFParser::parseRelocInfo(ParseContext &Ctx, DataExtractor &Extractor,
+                                uint64_t RelocInfoStart,
----------------
yonghong-song wrote:
> eddyz87 wrote:
> > MaskRay wrote:
> > > @ast @yonghong-song BTF relocations seem undocumented on https://www.kernel.org/doc/html/latest/bpf/btf.html#btf-ext-section ?
> > I will add necessary documentation.
> @eddyz87 Let us add relocation documentation in https://github.com/torvalds/linux/blob/master/Documentation/bpf/llvm_reloc.rst
> and add a reference in https://www.kernel.org/doc/html/latest/bpf/btf.html#btf-ext-section inst
> Thanks!
Working on it, submitted patch [[ https://lore.kernel.org/bpf/20230824230102.2117902-1-eddyz87@gmail.com/ | here ]].

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150079