[PATCH] D109217: [llvm-dwarfdump] Fix unsigned overflow when calculating stats

Thorsten via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 9 14:00:38 PDT 2021


tschuett added a comment.

That is fine be me. I guess the point is a save way to collect statistics and give guidance to users when the results could be bad, in release and debug mode. I would argue that saturating integers are different and maybe more precise solution than going from uint32_t to uint64_t ...


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

https://reviews.llvm.org/D109217



More information about the llvm-commits mailing list