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

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 18 05:52:39 PDT 2020


aaron.ballman added a comment.

In D78508#2041432 <https://reviews.llvm.org/D78508#2041432>, @hans wrote:

> Doing special handling for SIZE_T, as opposed to long in general, on Windows as Richard suggests seems reasonable to me.


I would be fine with this so long as it doesn't allow `%zu` to match an `unsigned int`, etc. `SIZE_T` matching a `%z` seems reasonable to me.


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