On Fri, Oct 5, 2012 at 10:25 PM, Ted Kremenek <span dir="ltr"><<a href="mailto:kremenek@apple.com" target="_blank" class="cremed">kremenek@apple.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: kremenek<br>
Date: Sat Oct  6 00:25:43 2012<br>
New Revision: 165361<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=165361&view=rev" target="_blank" class="cremed">http://llvm.org/viewvc/llvm-project?rev=165361&view=rev</a><br>
Log:<br>
Place warn_impcast_different_enum_types in a new warning group, -Wenum-conversion, that is a subgroup of -Wconversion.<br></blockquote><div><br></div><div>Ted, it doesn't look like this patch actually added -Wenum-conversion as a subgroup of -Wconversion, and we had regressions from code that used -Wno-conversion and started to get -Wenum-conversion warnings.</div>
<div><br></div><div>I've fixed this in r165440, could you take a look and make sure that looks right? Also, a test case for -Wno-conversion turning it off might not go amiss... I wonder if we could generated test cases for all of the set logic we currently have and then just have nice simple diffs when we change it.... Maybe not worth it.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Modified:<br>
    cfe/trunk/include/clang/Basic/DiagnosticGroups.td<br>
    cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td<br>
<br>
Modified: cfe/trunk/include/clang/Basic/DiagnosticGroups.td<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/DiagnosticGroups.td?rev=165361&r1=165360&r2=165361&view=diff" target="_blank" class="cremed">http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/DiagnosticGroups.td?rev=165361&r1=165360&r2=165361&view=diff</a><br>

==============================================================================<br>
--- cfe/trunk/include/clang/Basic/DiagnosticGroups.td (original)<br>
+++ cfe/trunk/include/clang/Basic/DiagnosticGroups.td Sat Oct  6 00:25:43 2012<br>
@@ -33,6 +33,7 @@<br>
 def SignConversion : DiagGroup<"sign-conversion">;<br>
 def BoolConversion : DiagGroup<"bool-conversion">;<br>
 def IntConversion : DiagGroup<"int-conversion">;<br>
+def EnumConversion : DiagGroup<"enum-conversion">;<br>
 def NonLiteralNullConversion : DiagGroup<"non-literal-null-conversion">;<br>
 def NullConversion : DiagGroup<"null-conversion">;<br>
 def ImplicitConversionFloatingPointToBool :<br>
<br>
Modified: cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td?rev=165361&r1=165360&r2=165361&view=diff" target="_blank" class="cremed">http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td?rev=165361&r1=165360&r2=165361&view=diff</a><br>

==============================================================================<br>
--- cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td (original)<br>
+++ cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td Sat Oct  6 00:25:43 2012<br>
@@ -1970,7 +1970,7 @@<br>
   InGroup<StringConversion>, DefaultIgnore;<br>
 def warn_impcast_different_enum_types : Warning<<br>
   "implicit conversion from enumeration type %0 to different enumeration type "<br>
-  "%1">, InGroup<DiagGroup<"conversion">>;<br>
+  "%1">, InGroup<EnumConversion>;<br>
 def warn_impcast_bool_to_null_pointer : Warning<<br>
     "initialization of pointer of type %0 to null from a constant boolean "<br>
     "expression">, InGroup<BoolConversion>;<br>
<br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu" class="cremed">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank" class="cremed">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</blockquote></div><br></div>