dcoughlin accepted this revision. dcoughlin added a comment. This revision is now accepted and ready to land.
Content-wise, LGTM. There is a style nit inline. Also, can you avoid reformatting the lines that haven't changed? This will help preserve the history of the file and make it clear what changes are related to your intended change in functionality. ================ Comment at: lib/Analysis/AnalysisDeclContext.cpp:308 +BodyFarm *AnalysisDeclContextManager::getBodyFarm() { + if (BdyFrm == nullptr) + BdyFrm = new BodyFarm(ASTCtx, Injector.get()); ---------------- Nit: Style-wise this is idiomatically written as: ``` if (!BdyFrm) ``` https://reviews.llvm.org/D39208 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits