JonasToth accepted this revision. JonasToth added a comment. This revision is now accepted and ready to land.
LGTM ================ Comment at: lib/Analysis/ExprMutationAnalyzer.cpp:381 +FunctionParmMutationAnalyzer::findMutation(const ParmVarDecl *Parm) { + const auto Memoized = Results.find(Parm); + if (Memoized != Results.end()) ---------------- shuaiwang wrote: > JonasToth wrote: > > Please spell out the type here > This type is a bit cumbersome to spell out as it's an iterator. I feel it's > fine to keep it auto. Oh true, iterator can be `auto`ed I would say ;) Repository: rC Clang https://reviews.llvm.org/D52008 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits