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

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 23 10:10:00 PST 2017

Previous message: [PATCH] D28967: [PGO] Add option to view CFG with profile after profile annotation
Next message: [PATCH] D28998: [lld/LTO] Add a flag to use the new PM during LTO
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

ruiu added a comment.

I'd probably just cast to `unsigned long` and use `lu`. Using sprintf with `(size_t)COFF::NameSize` is indeed better.


https://reviews.llvm.org/D28991

Previous message: [PATCH] D28967: [PGO] Add option to view CFG with profile after profile annotation
Next message: [PATCH] D28998: [lld/LTO] Add a flag to use the new PM during LTO
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list