[cfe-commits] r154667 - in /cfe/trunk: lib/Sema/SemaPseudoObject.cpp test/SemaObjCXX/property-reference.mm

Douglas Gregor dgregor at apple.com
Fri Apr 13 09:08:41 PDT 2012


On Apr 13, 2012, at 9:03 AM, jahanian wrote:

> 
> On Apr 13, 2012, at 9:01 AM, Douglas Gregor wrote:
> 
>> 
>> On Apr 13, 2012, at 8:57 AM, jahanian wrote:
>> 
>>> 
>>> On Apr 13, 2012, at 8:53 AM, Douglas Gregor wrote:
>>> 
>>>> Author: dgregor
>>>> Date: Fri Apr 13 10:53:08 2012
>>>> New Revision: 154667
>>>> 
>>>> URL: http://llvm.org/viewvc/llvm-project?rev=154667&view=rev
>>>> Log:
>>>> super and class property reference expressions don't need to be
>>>> rebuilt. Fixes <rdar://problem/11052352>. 
>>> 
>>> Please add a test for 'super' reference as well.
>> 
>> 
>> 'super' doesn't actually happen, because you can't have 'super.foo' within a template. 
> 
> Good point. Is this bug triggered inside a template only? If not, can we have a non-template test involving 'super' too?
> My concern is the more frequently used non-template cases.


This code is only triggered when we're performing a TreeTransform, which in turn only happens when we're dealing with templates (so 'super' isn't valid).

	- Doug



More information about the cfe-commits mailing list