george.karpenkov added inline comments.

================
Comment at: lib/StaticAnalyzer/Core/Z3ConstraintManager.cpp:928
+  bool isModelFeasible() override {
+    return Solver.check() != Z3_L_FALSE;
+  }
----------------
george.karpenkov wrote:
> The semantics of this method is incorrect. It should return a tri-value 
> somehow (e.g. `Optional<bool>`, and then higher-level logic in visitor should 
> decide what to do with it.)
We could also use `ConditionTruthVal` for this purpose.


https://reviews.llvm.org/D45517



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to