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

Reply via email to