[cfe-commits] r58609 - in /cfe/trunk: include/clang/AST/ASTContext.h lib/AST/ASTContext.cpp

Chris Lattner clattner at apple.com
Mon Nov 3 07:32:29 PST 2008


On Nov 3, 2008, at 6:12 AM, Douglas Gregor wrote:
> URL: http://llvm.org/viewvc/llvm-project?rev=58609&view=rev
> Log:
> Connect ASTContext to TargetInfo when determining the size_t,  
> ptrdiff_t, and wchar_t types. Fixes recent breakage on Linux.

Yay for reduction in FIXMEs :)

> +++ cfe/trunk/include/clang/AST/ASTContext.h Mon Nov  3 08:12:49 2008
> @@ -15,6 +15,7 @@
> #define LLVM_CLANG_AST_ASTCONTEXT_H
>
> #include "clang/Basic/LangOptions.h"
> +#include "clang/Basic/TargetInfo.h"
> #include "clang/AST/Builtins.h"
> #include "clang/AST/DeclBase.h"
> #include "clang/AST/Type.h"
> @@ -319,6 +319,8 @@
>  void setBuiltinVaListType(QualType T);
>  QualType getBuiltinVaListType() const { return BuiltinVaListType; }
>
> +  QualType getFromTargetType(TargetInfo::IntType Type) const;

How ugly would it be to prototype this as taking an unsigned instead  
of the enum type?  That would allow us to avoid the #include.

-Chris



More information about the cfe-commits mailing list