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.

Sign up for GitHub

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

missing warning for ambiguous else #11981

Closed

llvmbot opened this issue Dec 18, 2011 · 4 comments

Labels

bugzilla clang:frontend

Collaborator

llvmbot commented Dec 18, 2011


Bugzilla Link	11609
Resolution	FIXED
Resolved on	Dec 22, 2011 17:26
Version	unspecified
OS	Linux
Reporter	LLVM Bugzilla Contributor
CC	@nico

Extended Description

gcc will warn on

void g(int);
void f(int a, int b) {
if (a)
if (b)
g(1);
else
g(2);
g(3);
}

but clang doesn't.

Contributor

nico commented Dec 20, 2011

possible patch
Based on http://drdobbs.com/blogs/cpp/231602010

I didn't evaluate the usefulness of this warning yet.

Contributor

nico commented Dec 20, 2011

with actual test

Contributor

nico commented Dec 20, 2011

Sent out for review: http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20111219/050474.html

Contributor

nico commented Dec 23, 2011

r147202

llvmbot transferred this issue from llvm/llvm-bugzilla-archive

This issue was closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment