[all-commits] [llvm/llvm-project] 6a1f8e: [clang-tidy] Support begin/end free functions in m...

Chris Cotter via All-commits all-commits at lists.llvm.org
Sat Aug 5 14:08:37 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 6a1f8ef8a7aaefea80ef0bc7c6c462a96215b50e
      https://github.com/llvm/llvm-project/commit/6a1f8ef8a7aaefea80ef0bc7c6c462a96215b50e
  Author: Chris Cotter <ccotter14 at bloomberg.net>
  Date:   2023-08-05 (Sat, 05 Aug 2023)

  Changed paths:
    M clang-tools-extra/clang-tidy/modernize/LoopConvertCheck.cpp
    M clang-tools-extra/docs/ReleaseNotes.rst
    M clang-tools-extra/docs/clang-tidy/checks/modernize/loop-convert.rst
    M clang-tools-extra/test/clang-tidy/checkers/modernize/Inputs/loop-convert/structures.h
    M clang-tools-extra/test/clang-tidy/checkers/modernize/loop-convert-basic.cpp
    M clang-tools-extra/test/clang-tidy/checkers/modernize/loop-convert-negative.cpp
    M clang-tools-extra/test/clang-tidy/checkers/modernize/loop-convert-reverse.cpp

  Log Message:
  -----------
  [clang-tidy] Support begin/end free functions in modernize-loop-convert

The modernize-loop-convert check will now match iterator based loops
that call the free functions 'begin'/'end', as well as matching the
free function 'size' on containers.

Test plan: Added unit test cases matching free function calls on
containers, and a single negative test case for length() which is not
supported.

Reviewed By: PiotrZSL

Differential Revision: https://reviews.llvm.org/D140760




More information about the All-commits mailing list