[PATCH] D26922: [ObjC++] Don't enter a C++ declarator context when the current context is an Objective-C declaration
Alex Lorenz via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Nov 30 03:02:42 PST 2016
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
More information about the cfe-commits
mailing list