[PATCH] D66526: WIP: [utils] Add the llvm-locstats tool

[Djordje Todorovic via Phabricator via llvm-commits]

djtodoro marked an inline comment as done.
djtodoro added inline comments.


================
Comment at: llvm/utils/llvm-locstats/llvm-locstats.py:175
+    variables_total_locstats = \
+      json_parsed['total variables procesed by location statistics']
+    variables_scope_bytes_covered = \
----------------
vsk wrote:
> djtodoro wrote:
> > vsk wrote:
> > > What's the difference between this ('total variables procesed by location statistics') and 'total vars procesed by location statistics' above?
> > I just followed the existing name convention within `llvm-dwarfdump` statistics. Reporting stats with "`variables"` applies to both local variables and formal parameters, `"vars"` applies only to local variables and `"formal parameters"` obviously applies only to formal parameters.
> Ok. Perhaps we can revisit that & pick a clearer naming convention later.
I agree with that.
Thanks a lot for the review and comments! I really appreciate it!


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

https://reviews.llvm.org/D66526