[PATCH] D66121: Debug Info: Nest Objective-C property function decls inside their container.

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 13 11:26:35 PDT 2019

rjmccall added inline comments.

Comment at: clang/lib/CodeGen/CGDebugInfo.h:406
+                         CGBuilderTy &Builder,
+                         const ObjCContainerDecl *CD = nullptr);
aprantl wrote:
> rjmccall wrote:
> > Why does this have to be an extra parameter?  The method's DC should be the container.
> ```
> @protocol BarProto
> @property struct Bar *bar;
> @end
> @interface Foo <BarProto>
> @end
> @implementation Foo {}
> @synthesize bar = _bar;
> @end
> ```
> The ObjCMethodDecl for the synthesized `bar` accessors is the method decl in the protocol `BarProto`, as there is no method decl put into the AST inside of `Foo`. Its DeclContext (lexical an regular) therefor is the Protocol, which is not what we want as the decl context for a synthesized method called `-[Foo setBar:]` in the debug info.
Wait, really?  That seems pretty unfortunate.  Should we synthesize a real implementation method?



More information about the cfe-commits mailing list