https://github.com/AaronBallman requested changes to this pull request.
I don't think we should play whack-a-mole trying to find all the places we could potentially print a large integer value and update the diagnostic call sites. I think this is something the diagnostics engine should handle when deciding how to render an integer so that the behavior is consistent across diagnostics. Otherwise we'll have to do the same dance for `note_constexpr_new_too_large`, `err_attribute_size_too_large`, and so on. https://github.com/llvm/llvm-project/pull/211785 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
