[PATCH] D41458: [libc++][C++17] Elementary string conversions for integral types
Zhihao Yuan via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sat Mar 17 22:04:08 PDT 2018
lichray marked an inline comment as not done.
lichray added inline comments.
Comment at: include/support/itoa/itoa.h:29
> The `UINT64_C` and `UINT32_C` macros are non-standard, so I would rather not use them in a header.
> Additionally, the compiler should correctly convert the integer type to the array's element type, no?
They are from C standard, and I included the corresponding C header. Dropping them gave me `-Wimplicitly-unsigned-literal` warning.
More information about the cfe-commits