[PATCH] get rid of annoying windows warning C4146 in APInt.h ( no semantics change )

Peng Cheng gm4cheng at gmail.com
Tue Feb 26 19:55:41 PST 2013


-1ULL has created windows warning C4146 when including this header.

Changing -1ULL to UINT64_MAX fixes the warning, while without changing
semantics.  It also could be cheaper because there is no need for
truncation to 64bit if ULL is more than 64bit long in some machine.

See fix in patch attached.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130226/1fd016d0/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: APInt.patch
Type: application/octet-stream
Size: 1721 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130226/1fd016d0/attachment.obj>


More information about the llvm-commits mailing list