[PATCH] D75063: [analyzer] StdLibraryFunctionsChecker: Add NotNull Arg Constraint
Balázs Benics via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Mar 3 09:40:04 PST 2020
steakhal added inline comments.
================
Comment at: clang/lib/StaticAnalyzer/Checkers/StdLibraryFunctionsChecker.cpp:684
+ .Case({
+ ReturnValueCondition(LessThanOrEq, ArgNo(2)),
+ })
----------------
Two lines below you are using the `{0U}` initialization syntax, and here the simple constructor call syntax.
Shouldn't we pick one?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75063/new/
https://reviews.llvm.org/D75063
More information about the cfe-commits
mailing list