[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