[PATCH] D39428: [Analyzer] As suggested, use value storage for BodyFarm

Gábor Horváth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 31 13:29:42 PDT 2017


xazax.hun added a comment.

In https://reviews.llvm.org/D39428#912126, @george.karpenkov wrote:

> @xazax.hun I'm really not convinced:


Unfortunately, the LLVM codebase right now does not strictly follow the style guide. But clang-format puts the references next to the variable (or function in case of return type) names. Also if you check the code snippets in the coding standard: https://llvm.org/docs/CodingStandards.html#beware-unnecessary-copies-with-auto you can see that where we officially put the references.


https://reviews.llvm.org/D39428





More information about the cfe-commits mailing list