<div dir="ltr">Hi!<div><br></div><div style>I implemented a warning in parser as suggested here <a href="http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20121217/159766.html">http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20121217/159766.html</a>, e.g. warn in the following case</div>

<div style>if () {<br>} if () { // probably should have been 'else if'<br>} else {<br>}</div><div style><br></div><div style>Please review!</div><div style><br></div><div style>This diagnostic found two places in chromium and another one in firefox source base that are suspicious to have missed else.</div>

<div style><br></div><div style>By the way, there is a name clash between Parser::ParenParseOption::CompoundStmt and CompoundStmt from AST.  Maybe the former is worth renaming?</div></div>