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).


Yes, I agree that it's better to move the check to ShouldEnterDeclaratorScope, 
I will update the patch.

> 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?


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