[PATCH] D33844: [clang-tidy] terminating continue check

Alexander Kornienko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri May 11 07:30:25 PDT 2018


alexfh accepted this revision.
alexfh added a comment.
This revision is now accepted and ready to land.

In https://reviews.llvm.org/D33844#1095862, @koldaniel wrote:

> In https://reviews.llvm.org/D33844#1086565, @alexfh wrote:
>
> >
>
>
> Which comments do you think are still relevant?


Weird. When I was writing the comment, the code looked identical to the version from Mar 27. It may have been something wrong with my browser, with the Phabricator, or with the way I used it. Anyway, looks good now. Sorry for the confusion.

Do you need someone to commit the patch for you?


https://reviews.llvm.org/D33844





More information about the cfe-commits mailing list