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

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 20 18:45:45 PDT 2021


dblaikie added a comment.

In D109217#3009505 <https://reviews.llvm.org/D109217#3009505>, @djtodoro wrote:

> - addressing comments
>   - now the warning looks as follows:
>
>
>
>   "#call site DIEs": N (llvm-dwarfdump: warning: this field overflows),

Maybe we could render it symbolically and just say:

  "#call site DIEs": >= 9223372036854775807

But yeah, maybe the warning is more suitable, not sure - I'll leave it up to you folks to decide what's best.


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

https://reviews.llvm.org/D109217



More information about the llvm-commits mailing list