[PATCH] D113465: [llvm-dwarfdump][Statistics] Handle LTO cases with cross CU referencing

[Dimitrije Milošević via Phabricator via llvm-commits]

dmilosevic141 updated this revision to Diff 386130.
dmilosevic141 marked 7 inline comments as done.
dmilosevic141 edited the summary of this revision.
dmilosevic141 added a comment.

Thanks @Orlando!
As far as the example in the summary goes, it was meant to showcase cross CU referencing in general, hence why the variables in the concrete inlined instance tree had full location coverage. I've updated the summary by adding a concrete example of what it looks like when a variable in a concrete inlined instance tree has 0% location coverage, which is what this patch actually fixes.
I've also dropped the //std::unique_ptr// wrapper around //CrossCUReferencingDIELocationTy// as there really was no reason for it.


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

https://reviews.llvm.org/D113465

Files:
  llvm/test/tools/llvm-dwarfdump/X86/LTO_CCU_zero_loc_cov.ll
  llvm/tools/llvm-dwarfdump/Statistics.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D113465.386130.patch
Type: text/x-patch
Size: 17640 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20211110/21267f91/attachment.bin>