[PATCH] D96871: [llvm-dwarfdump][locstats] Unify handling of inlined vars with no loc

[Adrian Prantl via Phabricator via llvm-commits]

aprantl added a comment.

This definitely won't hurt, but it's a bit inconsistent. The NULL check is inside collectStatsRecursive(), but the CU check is outside of collectStatsForDie(). It would be nice to be consistent and have all functions handle all DIE types with potentially an early exit at the top rather sometimes having the burden to sanitize the input at the call site.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96871