LLVM Bugzilla is read-only and represents the historical archive of all LLVM issues filled before November 26, 2021. Use github to submit LLVM bugs

Bug 10448 - -Wall should include -Wsign-compare
Summary: -Wall should include -Wsign-compare
Status: RESOLVED WONTFIX
Alias: None
Product: clang
Classification: Unclassified
Component: Frontend (show other bugs)
Version: trunk
Hardware: All All
: P normal
Assignee: Unassigned Clang Bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-07-22 19:08 PDT by Ami Fischman
Modified: 2013-11-15 15:15 PST (History)
6 users (show)

See Also:
Fixed By Commit(s):

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ami Fischman 2011-07-22 19:08:42 PDT 
gcc's -Wall includes -Wsign-compare.  It'd be nice if clang did the same.
(today -Wall does not include -Wsign-compare)
Comment 1 Reid Kleckner 2013-11-15 15:15:50 PST 
I'm just repeating discussion from hazy memory, but I think the answer is that -Wsign-compare has a sufficiently high false positive rate that we don't want it on by default.  It's useful for other people who are trying to get close to warning parity with MSVC and gcc, but they have to pass the flag explicitly.