[lld] [llvm] [llvm-readobj][ELF] Alter JSON/LLVM output on note sections to allow for multiple notes per section in JSON (PR #96813)

James Henderson via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 18 00:10:19 PDT 2024


jh7370 wrote:

> It's unfortunate that this left the useless `Notesection` dictionary key in each element of the `NoteSections` list, rather than removing the useless dictionary layer consistent with the new `Notes` list.

Could you give an example of what you'd prefer the output to be? In the presence of potentially multiple note sections, each with multiple notes, I can't see any redundant layering.

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


More information about the llvm-commits mailing list