[PATCH] D33844: terminating continue check

Daniel Kolozsvari via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 22 07:51:38 PDT 2018


koldaniel added a comment.

In https://reviews.llvm.org/D33844#798900, @alexfh wrote:

> Please don't forget to add cfe-commits to `Subscribers:` when initially sending the patch for review. Otherwise nobody on the list will see it.
>
> Could you run the check on LLVM+Clang and post a summary of your findings here?


Hi,

Result of a clang-tidy run on LLVM+Clang with only misc-terminating-continue turned on:

1 Warning:
llvm/tools/clang/lib/Parse/ParseTentative.cpp:1678:7: warning: terminating 'continue' [misc-terminating-continue]

  continue;
  ^~~~~~~~~
  break


https://reviews.llvm.org/D33844





More information about the cfe-commits mailing list