NoQ added inline comments.
================ Comment at: lib/StaticAnalyzer/Checkers/CStringChecker.cpp:2317-2320 else if (isCPPStdLibraryFunction(FDecl, "copy")) evalFunction = &CStringChecker::evalStdCopy; else if (isCPPStdLibraryFunction(FDecl, "copy_backward")) evalFunction = &CStringChecker::evalStdCopyBackward; ---------------- Yes, indeed, this patch does [[ https://reviews.llvm.org/D55875#1336686 | break ]] modeling of `std::copy`, because, well, we *did* have explicit modeling of `std::copy`. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55873/new/ https://reviews.llvm.org/D55873 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits