This is an archive of the discontinued LLVM Phabricator instance.

[Clang] Rename `disable-noundef-analysis` flag to `-[no-]enable-noundef-analysis`
ClosedPublic

Authored by hyeongyukim on Feb 16 2022, 6:33 PM.

Details

Summary

This flag was previously renamed enable_noundef_analysis to
disable-noundef-analysis, which is not a conventional name. (Driver and
CC1's boolean options are using [no-] prefix)
As discussed at https://reviews.llvm.org/D105169, this patch reverts its
name to [no-]enable_noundef_analysis and enables noundef-analysis as
default.

Diff Detail

Event Timeline

hyeongyukim created this revision.Feb 16 2022, 6:33 PM
hyeongyukim requested review of this revision.Feb 16 2022, 6:33 PM
MaskRay accepted this revision.Feb 17 2022, 11:24 PM

Thanks!

This revision is now accepted and ready to land.Feb 17 2022, 11:24 PM

enable_noundef_analysis

-enable-noundef-analysis

If there is a need for an alias -disable-noundef-analysis, we can add it later.

This revision was landed with ongoing or failed builds.Feb 18 2022, 12:02 AM
This revision was automatically updated to reflect the committed changes.