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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
