[PATCH] D61827: [clang-tidy] modernize-loop-convert: impl const cast iter

Torbjörn Klatt via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 13 13:52:48 PDT 2019


torbjoernk updated this revision to Diff 199326.
torbjoernk edited the summary of this revision.
torbjoernk added a comment.
Herald added subscribers: jdoerfert, jfb.

Fixed the issues pointed out by Don Hinton and added note on OpenMP to the check's docs as suggested by Roman Lebedev.

Also fixed a small typo in a comment within the tests.

Side note: Somehow, the various failing tests I had previously where due to the fact that `make check-clang-tools` was picking my system-installed clang-tidy-8 and not the one I was building... o.O


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

https://reviews.llvm.org/D61827

Files:
  clang-tools-extra/clang-tidy/modernize/LoopConvertCheck.cpp
  clang-tools-extra/docs/clang-tidy/checks/modernize-loop-convert.rst
  clang-tools-extra/test/clang-tidy/modernize-loop-convert-basic.cpp
  clang-tools-extra/test/clang-tidy/modernize-loop-convert-extra.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D61827.199326.patch
Type: text/x-patch
Size: 5352 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190513/d3cbb8da/attachment-0001.bin>


More information about the cfe-commits mailing list