[PATCH] D26691: [analyzer] Run clang-format and fix style
Dominic Chen via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Dec 2 13:48:57 PST 2016
ddcc added inline comments.
================
Comment at: lib/StaticAnalyzer/Core/RangeConstraintManager.cpp:459
// Notice that the lower bound is greater than the upper bound.
- RangeSet New = GetRange(St, Sym).Intersect(getBasicVals(), F, Upper, Lower);
+ RangeSet New = getRange(St, Sym).Intersect(getBasicVals(), F, Upper, Lower);
return New.isEmpty() ? nullptr : St->set<ConstraintRange>(Sym, New);
----------------
zaks.anna wrote:
> We should use lower case function names.
Are you saying more functions should be changed to lowercase (e.g. intersect)? Or that `getRange` should be `getrange`?
https://reviews.llvm.org/D26691
More information about the cfe-commits
mailing list