r179108 - Objective-C: This patch fixes a none-issuance of warning

[jahanian]

On Apr 9, 2013, at 1:47 PM, Douglas Gregor <dgregor at apple.com> wrote:

> 
> On Apr 9, 2013, at 10:52 AM, Fariborz Jahanian <fjahanian at apple.com> wrote:
> 
>> Author: fjahanian
>> Date: Tue Apr  9 12:52:29 2013
>> New Revision: 179108
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=179108&view=rev
>> Log:
>> Objective-C: This patch fixes a none-issuance of warning
>> when result type of protocol property and getter method
>> differ by fixing a more serious problem. When a forward
>> protocol declaration comes between its definition and
>> its use in class protocol list, the forward protocol
>> ast was being used in building the protocol list.
>> // rdar://12522752
> 
> Why is this a problem? The protocol list could have a reference to the protocol declaration that was actually visible, and the callers that need definitions should use getDefinition().

When we lookup the property in class’s protocol list, we hit forward declarations in the list. It is better to fix the list in one place rather than everywhere we need to look into
the list. Does this cause a problem for modules? If so, I can move this to where property is looked up. But I think this is a better fix.

- Fariborz


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130409/34d35046/attachment.html>