labath added a comment.

The patch looks pretty good. A reasonable way to test this would be again via 
`lldb-test object-file` . The command dumps the "unified section list" of the 
module, so if the debug info sections show up there, you know the symbol vendor 
has done it's job. You can look at 
`test/Shell/ObjectFile/ELF/build-id-case.yaml` for inspiration.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72650



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to