[PATCH] D26644: [analyzer] Rename assumeWithinInclusiveRange*()

Devin Coughlin via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 14 16:02:42 PST 2016


dcoughlin accepted this revision.
dcoughlin added a comment.
This revision is now accepted and ready to land.

LGTM, other then some indentation issues for arguments and parameters after the rename. Please fix those and commit! And thanks for splitting this up.



================
Comment at: include/clang/StaticAnalyzer/Core/PathSensitive/ConstraintManager.h:103
+  virtual ProgramStateRef assumeInclusiveRange(ProgramStateRef State,
                                                      NonLoc Value,
                                                      const llvm::APSInt &From,
----------------
Nit: The indentation here (and elsewhere) is off after the rename.


https://reviews.llvm.org/D26644





More information about the cfe-commits mailing list