[PATCH] Fix to PR8880 (clang dies processing a for loop).

Serge Pavlov sepavloff at gmail.com
Thu Oct 24 10:32:35 PDT 2013


sepavloff added you to the CC list for the revision "Fix to PR8880 (clang dies processing a for loop).".

Due to statement expressions supported as GCC extension, it is possible
to put 'break' or 'continue' into a loop/switch statement but outside
its body, for example:

    for ( ; ({ if (first) { first = 0; continue; } 0; }); )

Such usage must be diagnosed as an error, GCC rejects it.
To recognize such patterns flags BreakScope and ContinueScope are
temporarily turned off while parsing condition expression.

http://llvm-reviews.chandlerc.com/D2018

Files:
  include/clang/Basic/DiagnosticSemaKinds.td
  include/clang/Parse/Parser.h
  include/clang/Sema/Scope.h
  lib/Parse/ParseStmt.cpp
  lib/Parse/Parser.cpp
  lib/Sema/Scope.cpp
  test/Parser/bad-control.c
  test/Sema/statements.c
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D2018.1.patch
Type: text/x-patch
Size: 10652 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20131024/766522be/attachment.bin>


More information about the cfe-commits mailing list