[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