[PATCH] D105884: [llvm-readobj] Display multiple function names for stack size entries

Owen Reynolds via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 13 03:34:03 PDT 2021


gbreynoo created this revision.
gbreynoo added reviewers: MaskRay, wolfgangp, grimar, rahmanl.
Herald added subscribers: rupprecht, emaste.
Herald added a reviewer: jhenderson.
gbreynoo requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

The current implementation of displaying .stack_size information presumes that each entry represents a single function but this is not always the case. For example with the use of ICF multiple functions can be represented with the same code, meaning that the address found in a .stack_size entry corresponds to multiple function symbols.
This change allows multiple function names to be displayed when appropriate.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D105884

Files:
  llvm/test/tools/llvm-readobj/ELF/stack-sizes-multiple-symbols.test
  llvm/test/tools/llvm-readobj/ELF/stack-sizes.test
  llvm/tools/llvm-readobj/ELFDumper.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D105884.358219.patch
Type: text/x-patch
Size: 27127 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210713/6890de03/attachment.bin>


More information about the llvm-commits mailing list