[PATCH] D25326: [StaticAnalyser] Don't merge different returns in ExplodedGraph

Devin Coughlin via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 10 08:30:24 PDT 2016


dcoughlin added a comment.

In https://reviews.llvm.org/D25326#565919, @danielmarjamaki wrote:

> In https://reviews.llvm.org/D25326#564584, @zaks.anna wrote:
>
> > Please, fix the style issues before committing.
>
>
> Would it be ok to run clang-format on some files to clean up the formatting? At least some minor changes like removing trailing spaces.


We generally try to avoid large-scale fixes of formatting and spacing issues if they are not on lines already modified by a patch. This makes it a bit easier to track down the original commit for a line and also reduces merge conflicts.


Repository:
  rL LLVM

https://reviews.llvm.org/D25326





More information about the cfe-commits mailing list