Summary:
In particular there's a common chain:
OpaqueValueExpr->PseudoObjectExpr->ObjCPropertyRefExpr->ObjCPropertyDecl
and we weren't handling the first two edges
Reviewers: dgoldman, kadircet
Subscribers: ilya-biryukov, MaskRay, jkorous, arphaman, jfb, usaxena95, cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D72494
void VisitObjCProtocolExpr(const ObjCProtocolExpr *OPE) {
Outer.add(OPE->getProtocol(), Flags);
}
+ void VisitOpaqueValueExpr(const OpaqueValueExpr *OVE) {
+ Outer.add(OVE->getSourceExpr(), Flags);
+ }
+ void VisitPseudoObjectExpr(const PseudoObjectExpr *POE) {
+ Outer.add(POE->getSyntacticForm(), Flags);
+ }
};
Visitor(*this, Flags).Visit(S);
}
EXPECT_DECLS("ObjCPropertyRefExpr", "- (void)setX:(int)x");
Code = R"cpp(
- @interface Foo {}
- @property int x;
+ @interface I {}
+ @property(retain) I* x;
+ @property(retain) I* y;
@end
- void test(Foo *f) {
- [[f.x]] = 42;
+ void test(I *f) {
+ [[f.x]].y = 0;
}
)cpp";
- EXPECT_DECLS("ObjCPropertyRefExpr",
- "@property(atomic, assign, unsafe_unretained, readwrite) int x");
+ EXPECT_DECLS("OpaqueValueExpr", "@property(atomic, retain, readwrite) I *x");
Code = R"cpp(
@protocol Foo