[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
+    UINT64_C(1000),
+    UINT64_C(10000),
+    UINT64_C(100000),
EricWF wrote:
> 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.

  rCXX libc++


More information about the cfe-commits mailing list