[PATCH] D109217: [llvm-dwarfdump] Fix unsigned overflow when calculating stats
Djordje Todorovic via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Oct 8 02:37:13 PDT 2021
djtodoro added a comment.
In D109217#3049369 <https://reviews.llvm.org/D109217#3049369>, @aprantl wrote:
> I think this is reasonable, out of curiosity, would there be a benefit to using APInt? Probably not because 64 bits is already huge...
I guess we can use APInt as well, but the 64 bits seem enough for now. A challenge would be to teach all the front-ends how to parse the big numbers (>64bits), but it is achievable.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D109217/new/
https://reviews.llvm.org/D109217
More information about the llvm-commits
mailing list