[PATCH] D117293: enable noundef analysis with -fsanitize-memory-param-retval

This revision was automatically updated to reflect the committed changes. Closed by commit rG0f93448235fe: enable noundef analysis with -fsanitize-memory-param-retval (authored by kda). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117293/new/ https

[PATCH] D117293: enable noundef analysis with -fsanitize-memory-param-retval

vitalybuka added a comment. To clarify: LGTM as-is, I don't expect any changes. 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

[PATCH] D117293: enable noundef analysis with -fsanitize-memory-param-retval

kda added a comment. In D117293#3243986 , @vitalybuka wrote: > In D117293#3243772 , @vitalybuka > wrote: > >> Just noticed, -enable-noundef-analysis is missing tests undef >> llvm/llvm-project/clang/test/Driver

[PATCH] D117293: enable noundef analysis with -fsanitize-memory-param-retval

vitalybuka added a comment. In D117293#3244668 , @kda wrote: > In D117293#3243735 , @vitalybuka > wrote: > >> Nice! >> This completes the work found in: >> >> It's too late for this set of patches, but phabr

[PATCH] D117293: enable noundef analysis with -fsanitize-memory-param-retval

kda added a comment. In D117293#3243735 , @vitalybuka wrote: > Nice! > >>> This completes the work found in: > > It's too late for this set of patches, but phabricator supports "Depends on" > comments (I guess only of the first patch upload), and manua

[PATCH] D117293: enable noundef analysis with -fsanitize-memory-param-retval

vitalybuka added a comment. In 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

[PATCH] D117293: enable noundef analysis with -fsanitize-memory-param-retval

vitalybuka added a comment. 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) Repository: rG LLVM Github Monorepo CHA

[PATCH] D117293: enable noundef analysis with -fsanitize-memory-param-retval

vitalybuka accepted this revision. vitalybuka added a comment. This revision is now accepted and ready to land. Nice! >> This completes the work found in: It's too late for this set of patches, but phabricator supports "Depends on" comments (I guess only of the first patch upload), and manual

[PATCH] D117293: enable noundef analysis with -fsanitize-memory-param-retval

kda updated this revision to Diff 399937. kda added a comment. add missing files. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117293/new/ https://reviews.llvm.org/D117293 Files: clang/include/clang/Driver/Options.td clang/test/CodeGen/attr-n

[PATCH] D117293: enable noundef analysis with -fsanitize-memory-param-retval

kda created this revision. kda added a reviewer: vitalybuka. Herald added a subscriber: dang. kda requested review of this revision. Herald added projects: clang, Sanitizers. Herald added subscribers: Sanitizers, cfe-commits. Enable noundef analysis (-enable-noundef-analysis) via the -fsanitize-m