[PATCH] D81923: [clang-tidy] Add modernize-use-ranges check.
Eugene Zelenko via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jun 16 06:37:21 PDT 2020
Eugene.Zelenko added inline comments.
================
Comment at: clang-tools-extra/docs/ReleaseNotes.rst:149
+ Converts calls to std library algorithms that take a pair of iterators to use
+ the equivalent function in std::ranges.
+
----------------
Please highlight std::ranges with double back-ticks.
================
Comment at: clang-tools-extra/docs/clang-tidy/checks/modernize-use-ranges.rst:6
+
+Looks for calls to functions from the standard library algorithms that take a
+pair of iterators from a container or array and replaces them with the
----------------
Please synchronize with description in Release Notes.
================
Comment at: clang-tools-extra/docs/clang-tidy/checks/modernize-use-ranges.rst:8
+pair of iterators from a container or array and replaces them with the
+equivalent function from the c++20 ranges library.
+
----------------
C++20.
================
Comment at: clang-tools-extra/docs/clang-tidy/checks/modernize-use-ranges.rst:21
+
+As this check uses the c++20 ranges library this will only run on translation
+unit thats use ``-std=c++20``.
----------------
C++20.
================
Comment at: clang-tools-extra/docs/clang-tidy/checks/modernize-use-ranges.rst:22
+As this check uses the c++20 ranges library this will only run on translation
+unit thats use ``-std=c++20``.
+
----------------
Single back-ticks for command-line options.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D81923/new/
https://reviews.llvm.org/D81923
More information about the cfe-commits
mailing list