On Jan 19, 2011, at 8:29 AM, Douglas Gregor wrote:

> 
> On Jan 19, 2011, at 2:06 AM, John McCall wrote:
> 
>> Author: rjmccall
>> Date: Wed Jan 19 04:06:00 2011
>> New Revision: 123817
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=123817&view=rev
>> Log:
>> Change the canonical representation of array types to store qualifiers on the
>> outermost array types and not on the element type.  Move the CanonicalType
>> member from Type to ExtQualsTypeCommonBase;  the canonical type on an 
>> ExtQuals
>> node includes the qualifiers on the ExtQuals.  Assorted optimizations enabled
>> by this change.
>> 
>> getQualifiers(), hasQualifiers(), etc. should all now implicitly look through
>> array types.
> 
> Nifty. I only saw a 0.8% speedup when parsing Cocoa.h after applying this 
> change (I was hoping for more), but this is definitely a nice cleanup.

Yeah, I was a little disappointed, too.

John.
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to