Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

-Wall should include -Wsign-compare #10820

Closed
llvmbot opened this issue Jul 23, 2011 · 1 comment
Closed

-Wall should include -Wsign-compare #10820

llvmbot opened this issue Jul 23, 2011 · 1 comment
Labels
bugzilla Issues migrated from bugzilla clang:frontend Language frontend issues, e.g. anything involving "Sema" wontfix Issue is real, but we can't or won't fix it. Not invalid

Comments

@llvmbot
Copy link
Collaborator

llvmbot commented Jul 23, 2011

Bugzilla Link 10448
Resolution WONTFIX
Resolved on Nov 15, 2013 15:15
Version trunk
OS All
Reporter LLVM Bugzilla Contributor
CC @AnnaZaks,@rjmccall,@rnk,@seanm

Extended Description

gcc's -Wall includes -Wsign-compare. It'd be nice if clang did the same.
(today -Wall does not include -Wsign-compare)
@rnk
Copy link
Collaborator

rnk commented Nov 15, 2013

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.

@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 3, 2021
@Quuxplusone Quuxplusone added the wontfix Issue is real, but we can't or won't fix it. Not invalid label Jan 20, 2022
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bugzilla Issues migrated from bugzilla clang:frontend Language frontend issues, e.g. anything involving "Sema" wontfix Issue is real, but we can't or won't fix it. Not invalid
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@rnk @Quuxplusone @llvmbot