[PATCH] D79431: [analyzer] StdLibraryFunctionsChecker: Add better diagnostics
Kristóf Umann via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Aug 13 04:55:37 PDT 2020
Szelethus requested changes to this revision.
Szelethus added a comment.
This revision now requires changes to proceed.
Tests c:
I'm still not a huge fan of the warning message. Now it describes //what kind// of constraint was violated, but not //how// (too large? too small?). Also, while I respect that you didn't prefer my suggestion, I think the warning message there is still a lot more welcoming, even if it doesn't convey more information. But, I don't necessarily insist on it.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D79431/new/
https://reviews.llvm.org/D79431
More information about the cfe-commits
mailing list