[PATCH] D135419: [readobj] Make JSON output consistent for Other flags

[Paul Kirth via Phabricator via llvm-commits]

paulkirth created this revision.
paulkirth added reviewers: jhenderson, phosek, leonardchan.
Herald added subscribers: s.egerton, simoncook, emaste.
Herald added a project: All.
paulkirth requested review of this revision.
Herald added subscribers: llvm-commits, pcwang-thead, MaskRay.
Herald added a project: LLVM.

Currently in the LLVM/JSON output, the Other field can either be an
integer(when it is 0) or an object/list for other cases. This patch makes the
formatting consistent for all value ranges. This is particularly important for
machine readable output in JSON, as it makes handling the Other field a special
case

Depends on D134843 <https://reviews.llvm.org/D134843>


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D135419

Files:
  llvm/test/tools/llvm-readobj/ELF/dyn-symbols-size-from-hash-table.test
  llvm/test/tools/llvm-readobj/ELF/dyn-symbols.test
  llvm/test/tools/llvm-readobj/ELF/sections-ext.test
  llvm/test/tools/llvm-readobj/ELF/symbol-shndx.test
  llvm/test/tools/llvm-readobj/ELF/symbol-visibility.test
  llvm/test/tools/llvm-readobj/ELF/symbols.test
  llvm/test/tools/llvm-readobj/ELF/symtab-shndx.test
  llvm/test/tools/llvm-readobj/ELF/versym-invalid.test
  llvm/tools/llvm-readobj/ELFDumper.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D135419.465951.patch
Type: text/x-patch
Size: 21615 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20221007/db65613d/attachment.bin>