[PATCH] D146753: [llvm-readobj] Include section information in summary

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 24 00:53:10 PDT 2023


jhenderson added inline comments.


================
Comment at: llvm/test/tools/llvm-readobj/ELF/file-summary-json.test:20
+# SINGLE-NEXT:      "LoadName": "<Not found>",
+# SINGLE-NEXT:      "TotalSections": 3,
+# SINGLE-NEXT:      "TotalSize": 20,
----------------
TotalSections will be available in the file header already, so isn't this a bit superfluous?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146753



More information about the llvm-commits mailing list