[cfe-commits] r148142 - in /cfe/trunk: include/clang/AST/Decl.h include/clang/Sema/Sema.h lib/AST/Decl.cpp lib/Sema/SemaChecking.cpp

[Chris Lattner]

On Jan 14, 2012, at 9:16 AM, Anna Zaks wrote:

>> 
>> Hi Anna,
>> 
>> Is there a reason not to just use Builtin ID's directly?  Why have another enum?
>> 
> 
> Good point. 
> The reason for the new enum is that it provides better readability: documents what the method handles; and in all the places where the value is used, makes it obvious that it's not just a single builtin. 
> On the other hand, it's probably more undesirable to grow FunctionDecl.. I'll patch to use the built-in ID.

Thanks Anna!

-Chris