[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