Hi! VS2010 now includes a stdint.h header. LLVM assumes that there is no stdint.h header and creates own types and macros. Because the macros in the stdint.h header are not protected, you get warnings at compile time if you first include llvm/Support/Datatypes.h and then stdint.h. The attached patch includes stdint.h depending on compiler version and therefore avoids these warnings. Kai -------------- next part -------------- An embedded and charset-unspecified text was scrubbed... Name: datatypes.diff URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20120114/629749b1/attachment.ksh>
Kai,> The attached patch includes stdint.h depending on compiler version and > therefore avoids these warnings.Generally the patch is ok, but will you please reformat it slightly - there is no need to add second level of nesting for _MSC_VER check. What if you just add the #elif case there? Thanks! -- With best regards, Anton Korobeynikov Faculty of Mathematics and Mechanics, Saint Petersburg State University