[PATCH] D65178: WIP: [DWARF] Collect statistics for DW_OP_entry_value

[Vedant Kumar via Phabricator via llvm-commits]

vsk added a comment.

In D65178#1600664 <https://reviews.llvm.org/D65178#1600664>, @djtodoro wrote:

> @vsk IIUC, you think the tool will be useful ? :)
>
> > I’ll set this patch aside for now.
>
> I think the goal of this patch is desirable for sure. In addition, I am not opposed to coexisting of this kind of stats within both of the //llvm-dwarfdump// and //llvm-locstats//, but that can be a point of discussion.
>
> > Maybe any relevant parts can be folded into the locstats tool later.
>
> Thanks! Those parts can be folded for sure! Just to mention, currently, there is an option in the //llvm-locstats// tool (--ignore-entry-values) that reports the debug location coverage by ignoring the debug entry values.


Thanks, that's perfect. I was looking for something just like 'llvm-locstats -only-formal-parameters [-ignore-entry-values]'. I'd love to see this functionality upstream eventually (maybe as a dwarfdump subtool?).


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

https://reviews.llvm.org/D65178