[PATCH] D53076: [analyzer] Enhance ConditionBRVisitor to write out more information

Umann Kristóf via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 23 04:49:01 PDT 2018


Szelethus added inline comments.


================
Comment at: include/clang/StaticAnalyzer/Core/PathSensitive/ConstraintManager.h:182
+  /// constraint being changed.
+  bool isChanged(const Stmt *Cond, StringRef Message) {
+    ConstraintMap::iterator I = Constraints.find(Cond);
----------------
I would not expect a function called `isChanged` to change the state of an object. Can you find a name that doesn't feel like it's a simple getter function?


https://reviews.llvm.org/D53076





More information about the cfe-commits mailing list