[cfe-commits] Patch to make clang compile and work on Windows using MSVC8
hartmut.kaiser at gmail.com
Sat Sep 1 15:56:09 PDT 2007
> >>> The compilation still produces a lot of warnings wrt to a
> >>> loss of precision while converting uint64_t's to
> >> uint32_t's. Is this a
> >>> known issue?
> >>> Do you intend to fix these?
> >> I am unaware of these (compiling clang on MacOS produces 0
> > Do you want me to fix these? At least I could try :-P
> That would be great :). I think that this is due to a
> warning that VC
> ++ produces that GCC doesn't by default. Most of these can be
> resolved by inserting the appropriate casts.
The attached patch fixes almost all of the warnings I mentioned. There are
two more warnings I didn't fix in the code:
warning C4146: unary minus operator applied to unsigned type, result
which I disabled in the project files and
warning C4715: 'some function name' : not all control paths return a
which is always caused by having an 'assert(0 && ...)' as the last statement
in some functions. These are still generated since I do not like to disable
them completely in the project files, and I don't know about any portable
fix for them.
Please note, this patch contains the two other patches I submitted
previously as well.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 17130 bytes
Desc: not available
More information about the cfe-commits