[PATCH] D108453: [Clang/Test]: Rename enable_noundef_analysis to disable-noundef-analysis and turn it off by default (2)
Juneyoung Lee via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sun Aug 22 23:55:58 PDT 2021
aqjune added a comment.
Hello all, I'm a messenger of @hyeongyukim's two patches that allow clang to turn on noundef analysis by default.
The motivation and background of these patches are described at https://reviews.llvm.org/D105169#2959464.
Here is a list of tests that has `RUN: %clang_cc1` replaced with `RUN: %clang_cc1 -disable-noundef-analysis`: F18629568: list.txt <https://reviews.llvm.org/F18629568>
They are updated to simply have the `-disable-noundef-analysis` because of the complicatedness of fixing them.
As described in https://reviews.llvm.org/D105169#2959464, a major concern with turning on this flag is that it requires a lot of tests to be updated, possibly raising conflicts with downstreams.
To avoid this, I'd like to receive requests for tests that need `-disable-noundef-analysis`. If the directories or tests are specified, I'll simply add the flag to the `RUN:` command.
This will reduce the opportunity of raising conflicts with downstreams.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D108453/new/
https://reviews.llvm.org/D108453
More information about the cfe-commits
mailing list