[clang] 5d110ed - Revert "[NFC] Update new warning to test"

Nico Weber via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 15 19:24:41 PST 2022


Author: Nico Weber
Date: 2022-02-15T22:23:26-05:00
New Revision: 5d110ed4cd475b9b70bc9faecfe4f0767740c5a3

URL: https://github.com/llvm/llvm-project/commit/5d110ed4cd475b9b70bc9faecfe4f0767740c5a3
DIFF: https://github.com/llvm/llvm-project/commit/5d110ed4cd475b9b70bc9faecfe4f0767740c5a3.diff

LOG: Revert "[NFC] Update new warning  to test"

This reverts commit 25cdf87b13eb990eb84d31211280f4b0d5d470b3.
125abb61f7ae reverted the patch.

It is incorrect to update this file when a flagless warning is added.
This test exists to make sure _all_ warnings are behind a flag.
The right fix is to put the new warning in a warning group (so that
it can be toggled with a flag), not to update the list here.

Added: 
    

Modified: 
    clang/test/Misc/warning-flags.c

Removed: 
    


################################################################################
diff  --git a/clang/test/Misc/warning-flags.c b/clang/test/Misc/warning-flags.c
index 1b65af4c8e3e5..a9e0a784c5c81 100644
--- a/clang/test/Misc/warning-flags.c
+++ b/clang/test/Misc/warning-flags.c
@@ -18,7 +18,7 @@ This test serves two purposes:
 
 The list of warnings below should NEVER grow.  It should gradually shrink to 0.
 
-CHECK: Warnings without flags (68):
+CHECK: Warnings without flags (67):
 
 CHECK-NEXT:   ext_expected_semi_decl_list
 CHECK-NEXT:   ext_explicit_specialization_storage_class
@@ -65,7 +65,6 @@ CHECK-NEXT:   warn_missing_dependent_template_keyword
 CHECK-NEXT:   warn_missing_whitespace_after_macro_name
 CHECK-NEXT:   warn_mt_message
 CHECK-NEXT:   warn_no_constructor_for_refconst
-CHECK-NEXT:   warn_no_support_for_eval_method_source_on_m32
 CHECK-NEXT:   warn_not_compound_assign
 CHECK-NEXT:   warn_objc_property_copy_missing_on_block
 CHECK-NEXT:   warn_objc_protocol_qualifier_missing_id


        


More information about the cfe-commits mailing list