[cfe-commits] PATCH: Adds -Wenum-compare to Clang

Chris Lattner clattner at apple.com
Wed Feb 16 16:58:10 PST 2011


+1

On Feb 16, 2011, at 4:01 PM, Ted Kremenek wrote:

> I like this warning to be on by default.
> 
> On Feb 15, 2011, at 10:41 PM, Chandler Carruth wrote:
> 
>> This patch implements most of -Wenum-compare in Clang. I know of at least on difference between this implementation and GCC: this is currently on by default for all languages. In GCC -Wenum-compare is only turned on with -Wall in C/ObjC, but is on by default in C++. I'm happy either leaving as is (and it always being on by default) or if someone has a suggestion for how to model GCC's behavior I'm happy to do that. There may be other differences in the nature of the implementation, I'm not familiar with GCC's. <warn_enum_compare.patch>_______________________________________________
>> cfe-commits mailing list
>> cfe-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
> 
> 
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits





More information about the cfe-commits mailing list