[PATCH] D117293: enable noundef analysis with -fsanitize-memory-param-retval
Kevin Athey via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jan 14 13:46:49 PST 2022
kda added a comment.
In D117293#3243986 <https://reviews.llvm.org/D117293#3243986>, @vitalybuka wrote:
> In D117293#3243772 <https://reviews.llvm.org/D117293#3243772>, @vitalybuka wrote:
>
>> Just noticed, -enable-noundef-analysis is missing tests undef llvm/llvm-project/clang/test/Driver/
>> Can you add few near maybe before llvm-project/clang/test/Driver/clang_f_opts.c:531 (just before discard-value-names tests)
>
> Actually noundef is not f_opt, and I am not sure how or do we need to test ImpliedByAnyOf in Driver.
The CodeGen tests cover these interactions.
When I was building this, I added the tests first (they failed), then I added the ImpliedByAnyOf one-liner, and the tests passed.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D117293/new/
https://reviews.llvm.org/D117293
More information about the cfe-commits
mailing list