[PATCH] VS2010 stdint.h

Project Infrastructure LLVM Dev List Archives
#1

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

datatypes.diff (1.26 KB)

#2

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!