dexonsmith accepted this revision. dexonsmith added a comment. This revision is now accepted and ready to land.
LGTM, with one style nitpick. ================ Comment at: clang/lib/CodeGen/CGObjCMac.cpp:4049-4051 + if (!OMD->getBody() || COMD->getReturnType() == OMD->getReturnType()) { + return I->second; + } ---------------- I think the LLVM style is to leave out these braces. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73208/new/ https://reviews.llvm.org/D73208 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits