arphaman added a comment.

In https://reviews.llvm.org/D26922#608815, @ahatanak wrote:

> Yes, I meant ParseDecl:5266. Reading the comment in 
> ShouldEnterDeclaratorScope, it seemed to me that it shouldn't return true in 
> this context (when parsing an objective-c).
>
> Also, the following code (which is not valid) crashes with ToT trunk,
>
>   @property (nonatomic) int OuterType::InnerType
>
>
> but compiles without any errors with your patch applied. Do you know why 
> clang doesn't error out?


Hmm, that's interesting, I have to investigate this. It looks like clang is 
creating a property named `InnerType` which is obviously invalid.


Repository:
  rL LLVM

https://reviews.llvm.org/D26922



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to