aprantl added inline comments.
================ Comment at: lib/CodeGen/CGDebugInfo.cpp:4239 + // Add methods to interface. + for (auto p : ObjCMethodCache) { + if (p.second.empty()) ---------------- JDevlieghere wrote: > dexonsmith wrote: > > Some comment for "p" here. > Fixed; I'll update the rest of the file in a follow-up to keep things > consistent. Unfortunately CFE doesn't seem to consistently use the LLVM coding style: A lot of CFE developers seem to prefer lower-case variable names. I don't know if that is a historic artifact or a conscious decision. https://reviews.llvm.org/D48241 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits