[PATCH] D77789: [llvm-dwarfdump][Stats] Clean up

Djordje Todorovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 28 06:56:22 PDT 2020


djtodoro updated this revision to Diff 260615.
djtodoro added a comment.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

-use a function-like syntax
-add the documentation

TODO: use the string literals to avoid `\\`.
(@aprantl I think we can use the string literals only in C++ part, since I think `r'..'`feature in Python was introduced in Python 3.x, but we still use Python 2.7 for the `llvm-locstats`)


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

https://reviews.llvm.org/D77789

Files:
  llvm/docs/CommandGuide/llvm-dwarfdump.rst
  llvm/test/DebugInfo/X86/dwarf-callsite-related-attrs.ll
  llvm/test/tools/llvm-dwarfdump/X86/locstats.ll
  llvm/test/tools/llvm-dwarfdump/X86/statistics-base-address.s
  llvm/test/tools/llvm-dwarfdump/X86/statistics-dwo.test
  llvm/test/tools/llvm-dwarfdump/X86/statistics-v3.test
  llvm/test/tools/llvm-dwarfdump/X86/statistics.ll
  llvm/test/tools/llvm-dwarfdump/X86/stats-dbg-callsite-info.ll
  llvm/test/tools/llvm-dwarfdump/X86/stats-inlining-multi-cu.ll
  llvm/test/tools/llvm-dwarfdump/X86/stats-inlining-single-cu.ll
  llvm/test/tools/llvm-dwarfdump/X86/stats-mulitple-cu-out-of-line.ll
  llvm/test/tools/llvm-dwarfdump/X86/stats-multiple-cu-members.ll
  llvm/test/tools/llvm-dwarfdump/X86/stats-multiple-cu-same-name.ll
  llvm/tools/llvm-dwarfdump/Statistics.cpp
  llvm/utils/llvm-locstats/llvm-locstats.py

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D77789.260615.patch
Type: text/x-patch
Size: 51181 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200428/0f5202cf/attachment.bin>


More information about the llvm-commits mailing list