[PATCH] D78508: [Clang] Allow long as size_t printf argument on 32-bit Windows platforms.

Hans Wennborg via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri May 15 10:51:38 PDT 2020


hans added a comment.

> clang issues a warning, leaving no good way to print SIZE_T (other than disabling warnings or adding useless casts)

I also don't think Clang should change here though. The warning is legit for code that cares about portability, and inserting a cast is probably the best fix for such code. For code that doesn't care, disabling the warning seems like the way to go.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D78508/new/

https://reviews.llvm.org/D78508





More information about the cfe-commits mailing list