[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 Jun 27 00:24:52 PDT 2024


jh7370 wrote:

I can't see a way of avoiding the extra indentation, due to the structure of the stuff being dumped. I'd ideally like to come up with a different title, so we don't have an inner and an outer Notes list title, because I have a feeling that this could make things somewhat tricky to parse, especially in lit tests. Perhaps the inner one could be something like "NoteEntries"?

I'll come to reviewing the code later.

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


More information about the llvm-commits mailing list