[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