[PATCH] D86044: [llvm-dwarfdump][test] Add a --statistics test for a DW_AT_artificial variable
    Fangrui Song via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Mon Aug 17 12:09:34 PDT 2020
    
    
  
MaskRay added a comment.
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.
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