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

Chandler Carruth chandlerc at google.com
Tue Feb 15 23:08:48 PST 2011


John asked on IRC for this to avoid warning on cross-anonymous-enum
comparisons, and that made a lot of sense. Done, and a new patch attached.

On Tue, Feb 15, 2011 at 10:41 PM, Chandler Carruth <chandlerc at google.com>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.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20110215/a924f1c6/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: warn_enum_compare.patch
Type: application/octet-stream
Size: 20555 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20110215/a924f1c6/attachment.obj>


More information about the cfe-commits mailing list