[llvm-commits] LLParser::ParseCmpPredicate TokError falls through

[David Tweed]

[Repost after sending to cfe-commits by mistake]

Hi,

LLParser::ParseCmpPredicate has two uses of TokError in a switch which, currently, fall through so an invalid cmp operand can becomes a valid, different parse. I don't really understand the big picture of the codeI can fix that either by doing "return TokError(...)" or adding a break. Using a break will result in a call of Lex.Lex() which there isn't in the return case. I can't tell any difference in testing, I think "return" is the right solution but can I just check that's not breaking some subtle parsing invariant? If it's the best solution, I'll roll up a patch and a test and commit.

Cheers,
Dave

-- IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium.  Thank you.