[cfe-commits] r68468 - in /cfe/trunk: include/clang/Basic/DiagnosticSemaKinds.td lib/Sema/SemaExpr.cpp test/SemaObjC/call-super-2.m

[steve naroff]

On Apr 6, 2009, at 8:23 PM, Fariborz Jahanian wrote:

>
> On Apr 6, 2009, at 5:19 PM, Daniel Dunbar wrote:
>
>> Hi Steve,
>>
>> One question about this patch. Will we be getting the "right" type  
>> for a casted super expression (i.e. the casted type)? If not, it  
>> seems like there is some possibility that we will end up  
>> miscompiling otherwise working code. Of course, this is probably  
>> very unlikely in practice...
>
> I think we should leave it as error and do not risk of mis-compiling  
> user source. In all cases that I
> filed radar against the projects, they were quick to respond  
> positively. But, if we want to warn,
> we should make sure that lookup rules follow closely to gcc's.
>

Hey Fariborz...please see my response to Daniel.

Thanks,

snaroff

> - Fariborz
>
>>