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

Doug Gregor doug.gregor at gmail.com
Mon Nov 3 07:57:02 PST 2008


On Mon, Nov 3, 2008 at 10:32 AM, Chris Lattner <clattner at apple.com> wrote:
>> +++ 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.

Not too ugly; getFromTargetType should be private anyway. Tweak committed.

  - Doug



More information about the cfe-commits mailing list