[llvm] [llvm-readobj][ELF] Implement JSON output for --dynamic-table (PR #95976)

James Henderson via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 25 01:14:12 PDT 2024


jh7370 wrote:

> > I'm thinking
> > https://github.com/llvm/llvm-project/blob/db9e9eabb7835bae4285a3f13c7cc7c985455e27/llvm/tools/llvm-readobj/ELFDumper.cpp#L2474
> > 
> > is the area that needs changing for the LLVM-style output.
> 
> Do we want to just jettison the brackets? Or were you thinking of something else?

Just jettisoning the brackets. I suspect whoever implemented this option got themselves confused and tried to format things like a list for individual entries (note that it's certainly possible to have multiple DT_NEEDED etc tags, so a "summarised" version might have a list of needed libraries, but in the case we're talking about, we're only dealing with each tag individually, and therefore only a single entry).

https://github.com/llvm/llvm-project/pull/95976


More information about the llvm-commits mailing list