aschwaighofer marked an inline comment as done. aschwaighofer added inline comments.
================ Comment at: clang/lib/CodeGen/CGObjCRuntime.h:217 + /// ProtocolPtrTy. + virtual llvm::Constant *GetOrEmitProtocol(const ObjCProtocolDecl *PD) = 0; + ---------------- aschwaighofer wrote: > rjmccall wrote: > > Can this name more closely parallel the "external" name, like > > `getOrEmitProtocolObject`? > > > > Also, I think we're incrementally lowercasing new method names. > This is an already existing method. I just moved it higher in the class > hierarchy to CGObjCRuntime from CGObjCMac so that it is callable from > getOrEmitProtocolObject. s/getOrEmitProtocolObject/emitObjCProtocolObject Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D77077/new/ https://reviews.llvm.org/D77077 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
