[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