[PATCH] D71091: Make sure that the implicit arguments for direct methods have been setup

Pierre Habouzit via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Dec 6 10:12:42 PST 2019


MadCoder added a comment.

just added all the context this time (-W)



================
Comment at: clang/lib/CodeGen/CGObjCMac.cpp:4030
                                       const ObjCContainerDecl *CD) {
-  auto I = DirectMethodDefinitions.find(OMD);
+  auto I = DirectMethodDefinitions.find(OMD->getCanonicalDecl());
   if (I != DirectMethodDefinitions.end())
----------------
jfb wrote:
> I'd rather have `auto *` or `Decl *` here.
this is a pair this doesn't work


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71091/new/

https://reviews.llvm.org/D71091





More information about the cfe-commits mailing list