Bowing to popular demand, reduce the "comma at end of enumerator list"

ExtWarn to an Extension. Let the broken code propagate!

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@107875 91177308-0d34-0410-b5e6-96231b3b80d8

diff --git a/include/clang/Basic/DiagnosticGroups.td b/include/clang/Basic/DiagnosticGroups.td
index 9b4c939a9530659d6efc74f6ed729024c7c924f3..8572c62e4bbe420e85795a4c2aa093bcd0257e16 100644 (file)
--- a/include/clang/Basic/DiagnosticGroups.td
+++ b/include/clang/Basic/DiagnosticGroups.td
@@ -38,7 +38,6 @@ def : DiagGroup<"disabled-optimization">;
 def : DiagGroup<"discard-qual">;
 def : DiagGroup<"div-by-zero">;
 def EmptyBody : DiagGroup<"empty-body">;
-def EnumeratorListComma : DiagGroup<"enumerator-list-comma">;
 def ExtraTokens : DiagGroup<"extra-tokens">;
 
 def FormatExtraArgs : DiagGroup<"format-extra-args">;

diff --git a/include/clang/Basic/DiagnosticParseKinds.td b/include/clang/Basic/DiagnosticParseKinds.td
index 734b414b5190007beae45299099f13ad45f7dd9a..63bd3f65a91face4bf4f657c6cdb736c653f6572 100644 (file)
--- a/include/clang/Basic/DiagnosticParseKinds.td
+++ b/include/clang/Basic/DiagnosticParseKinds.td
@@ -51,9 +51,9 @@ def ext_c99_variable_decl_in_for_loop : Extension<
   "variable declaration in for loop is a C99-specific feature">;
 def ext_c99_compound_literal : Extension<
   "compound literals are a C99-specific feature">;
-def ext_enumerator_list_comma : ExtWarn<
+def ext_enumerator_list_comma : Extension<
   "commas at the end of enumerator lists are a %select{C99|C++0x}0-specific "
-  "feature">, InGroup<EnumeratorListComma>;
+  "feature">;
 
 def ext_gnu_indirect_goto : Extension<
   "use of GNU indirect-goto extension">, InGroup<GNU>;

diff --git a/test/SemaCXX/overloaded-operator.cpp b/test/SemaCXX/overloaded-operator.cpp
index 15d6605d0258703acd19c89765e344fd694a9f90..24f7f66129ebd9b4564297e632701a8bf56b48ae 100644 (file)
--- a/test/SemaCXX/overloaded-operator.cpp
+++ b/test/SemaCXX/overloaded-operator.cpp
@@ -293,7 +293,7 @@ void circ() {
 
 // PR5360: Arrays should lead to built-in candidates for subscript.
 typedef enum {
-  LastReg = 23, // expected-warning{{commas at the end of enumerator lists are a C++0x-specific feature}}
+  LastReg = 23,
 } Register;
 class RegAlloc {
   int getPriority(Register r) {