NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land.
Fantastic, thank you! ================ Comment at: clang/lib/StaticAnalyzer/Core/ExprEngineCallAndReturn.cpp:586 ExplodedNodeSet dstCallEvaluated; + EvalCallOptions CallOpts; getCheckerManager().runCheckersForEvalCall(dstCallEvaluated, dstPreVisit, ---------------- If you want you can make it an optional argument of `runCheckersForEvalCall()`, like it's done in `defaultEvalCall()`. ================ Comment at: clang/test/Analysis/cxxctr-evalcall-analysis-order.cpp:34 +// CHECK-NEXT: PostCall (C::C) [CXXConstructorCall] \ No newline at end of file ---------------- No NEwLiNE aT EnD Of FiLE ================ Comment at: clang/test/Analysis/new-ctor-conservative.cpp:30 clang_analyzer_eval(s[0].x == 1); // expected-warning{{UNKNOWN}} + clang_analyzer_eval(s[1].x == 1); // expected-warning{{UNKNOWN}} } ---------------- Thx! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D82256/new/ https://reviews.llvm.org/D82256 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits