[llvm-commits] Windows compilation warnings
dimitry at andric.com
Thu Jul 15 18:05:57 CDT 2010
On 2010-07-15 20:45, Villmow, Micah wrote:
> There are a bunch of windows warnings, here is a patch that fixes some of them.
I assume these are warning fixes for the x64 build on Windows? Because
the Win32 ToT build just has some warnings about enum values being
truncated, and a few switch statements that only have a default case.
If these fixes are indeed for the Windows x64 build, I would propose not
to simply "cast away" some of those warnings, as they might hide real
There are *lot* of silent 64-to-32 bit truncations in the codebase, for
example where lengths (usually a size_t) are stuffed into plain int or
unsigned, and ideally these should all be checked to verify there is no
problem with that truncation.
That said, I will apply your patch to an x64 build, and see how many of
the (very many :) warnings go away.
More information about the llvm-commits