================ @@ -2808,27 +2825,63 @@ void ExprEngine::processBranch(const Stmt *Condition, std::tie(StTrue, StFalse) = *KnownCondValueAssumption; else { assert(!isa<ObjCForCollectionStmt>(Condition)); + // TODO: instead of this shortcut perhaps it would be better to "rejoin" + // the common execution path with + // StTrue = StFalse = PrevState; builder.generateNode(PrevState, true, PredN); builder.generateNode(PrevState, false, PredN); continue; ---------------- NagyDonat wrote:
I pushed a slightly tweaked variant of this and it does not break anything in the basic test suite (check-clang-analysis). Thanks for the suggestion! https://github.com/llvm/llvm-project/pull/109804 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits