[PATCH] D69027: [llvm-dwarfdump][Statistics] Fix calculation of OffsetToFirstDefinition

Djordje Todorovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 1 06:19:54 PDT 2019


djtodoro added a comment.

@krisb Thanks for summarizing the things!

I think as well that it will be good (and useful) to add new fields (buckets) for the local variables statistics with adjusted scope, and keep the existing buckets calculated against the enclosing scope. And then, we could add a new option into the llvm-locstats (http://llvm.org/docs//CommandGuide/llvm-locstats.html) to handle it properly.

Let us know what @aprantl and @vsk think about the ideas. :)


Repository:
  rL LLVM

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

https://reviews.llvm.org/D69027





More information about the llvm-commits mailing list