[PATCH] D86044: [llvm-dwarfdump][test] Add a --statistics test for a DW_AT_artificial variable

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 17 12:15:22 PDT 2020


dblaikie added a comment.

In D86044#2222156 <https://reviews.llvm.org/D86044#2222156>, @MaskRay wrote:

> In D86044#2222140 <https://reviews.llvm.org/D86044#2222140>, @dblaikie wrote:
>
>> Sorry, looking at the test now - is there any particular reason to test this case in particular? Looks like there's no particular statistics for artificial variables & I guess the statistics code doesn't look at this property one way or another?
>
> Statistics.cpp:343 has a different `DW_AT_artificial` use. If we change Statistics.cpp:564 to use NumArtificial in IsFunction case, a DW_AT_artificial variable will be significant.

Not sure I follow, sorry. Are you planning to add a new statistic?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86044



More information about the llvm-commits mailing list