[PATCH] D73270: [clang-tidy] Fix false positive in bugprone-infinite-loop

Dmitri Gribenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 24 08:24:08 PST 2020


gribozavr2 added a comment.

In D73270#1838883 <https://reviews.llvm.org/D73270#1838883>, @njames93 wrote:

> May not be one for this patch, but how does this check handle volatile loop variables and cases where modification isn't visible in the context e.g.


Should be OK, see "if (Var->getType().isVolatileQualified())" in the implementation. Feel free to submit another patch with more tests!


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D73270/new/

https://reviews.llvm.org/D73270





More information about the cfe-commits mailing list