[PATCH] Updated header includes for GCC 4.4

[llvmbot]

Bugzilla Link	3697
Resolution	FIXED
Resolved on	Mar 12, 2010 00:55
Version	unspecified
OS	Linux
Attachments	gcc-4.4 patch for clang
Reporter	LLVM Bugzilla Contributor

Extended Description

GCC 4.4's g++ is now even stricter about header inclusion than GCC 4.4, and clang does not currently compile out of the box. The attached patch includes the necessary headers (cstdio, stdint.h) to get clang to compile.

Note: this really affects all of clang, I'm guessing the "Basic" component. Should there be a "general" component?

[lattner]

Ok, but please add standard headers after llvm/clang headers. Instead of:

#include
#include "clang/Lex/HeaderMap.h"

Please use:

#include "clang/Lex/HeaderMap.h"
#include

etc, thanks!

[llvmbot]

Updated patch: follow pre-existing ordering of includes
Fixed patch attached. Apologies, the first few files I patched did not have system headers included, and so I didn't notice the local/system ordering convention.

[lattner]

No need to apologize! your patch looks great, applied here:
http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20090302/013333.html

Thanks!