[PATCH] D111184: [clang] Allow printing 64 bit ints in diagnostics

2021-10-18 Thread Arthur Eubanks via Phabricator via cfe-commits
aeubanks added a comment. In D84#3071422 , @dblaikie wrote: > Is there no diagnostic that could reasonably produce such a large value that > would be easy to test (without needing a giant binary, etc)? (seem surprising > to see this change without a

[PATCH] D111184: [clang] Allow printing 64 bit ints in diagnostics

2021-10-18 Thread David Blaikie via Phabricator via cfe-commits
dblaikie added a comment. Is there no diagnostic that could reasonably produce such a large value that would be easy to test (without needing a giant binary, etc)? (seem surprising to see this change without any test coverage) Like a 64 bit non-type template parameter or the like, perhaps? Re

[PATCH] D111184: [clang] Allow printing 64 bit ints in diagnostics

2021-10-05 Thread Arthur Eubanks via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGedfff2f8b043: [clang] Allow printing 64 bit ints in diagnostics (authored by aeubanks). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST AC

[PATCH] D111184: [clang] Allow printing 64 bit ints in diagnostics

2021-10-05 Thread Arthur Eubanks via Phabricator via cfe-commits
aeubanks created this revision. aeubanks requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Currently we're limited to 32 bit ints in diagnostics. With support for 4GB alignments coming soon, we need to report 4GB as the max alignment allowed