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 _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
