https://gcc.gnu.org/bugzilla/show_bug.cgi?id=51712
--- Comment #18 from Jonathan Nieder ---
That said, as mentioned in comment#15, the use of -fno-short-enums in the test
is not right. I'll try removing that and see if the test still passes tomorrow
(it should).
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=51712
Jonathan Nieder changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51712
--- Comment #13 from Jonathan Nieder jrnieder at gmail dot com 2012-10-16
07:55:56 UTC ---
Hi Kyrill,
(In reply to comment #11)
Adding the -fno-short-enums fixes the
extra warning generated by the arg = 0 comparison in pr51712.c
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51712
--- Comment #4 from Jonathan Nieder jrnieder at gmail dot com 2012-04-17
18:13:08 UTC ---
I am not convinced clang's heuristic is the right one. For example, the
following code trips clang's warning, but the test is still not redundant. The
main
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51712
--- Comment #2 from Jonathan Nieder jrnieder at gmail dot com 2012-03-26
20:12:17 UTC ---
Clang does not consider arg = FOO to be a comparison against 0.
| commit e3b159c0
| Author: Ted Kremenek kreme...@apple.com
| Date: Thu Sep 23 21:43:44
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51712
Bug #: 51712
Summary: -Wtype-limits should not trigger for types of
implementation-defined signedness
Classification: Unclassified
Product: gcc
Version: unknown
