[PATCH] D28991: [lld] Use %u instead of %d for uint32_t formatting

Joerg Sonnenberger via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jan 22 12:59:19 PST 2017


joerg added a comment.

Neither is correct. It should be casted either to uintmax_t and printed with %ju or PRIu32 be used. The fancy new formatting system might also work of course, but won't give you a C string.
That said, this should also be changed to use snprintf...


https://reviews.llvm.org/D28991





More information about the llvm-commits mailing list