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

[Aaron Ballman via Phabricator via cfe-commits]

aaron.ballman added a comment.

In D78508#2038793 <https://reviews.llvm.org/D78508#2038793>, @amccarth wrote:

> I'll re-iterate my opinion that the casts currently required to bypass the warnings are useful (e.g., if you ever want to port the code to 64-bit).  There are lots of places where the Windows API essentially required uncomfortable conversions, and we can't paper over all of them.  The way to get it right in all those other places is to be very aware of the underlying types.  Hiding an uncomfortable detail here and there seems like a disservice.


+1


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78508