[all-commits] [llvm/llvm-project] c96306: [clang-tidy] Fix `llvmlibc-inline-function-decl` f...

[AMS21 via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: c96306db2cad4cf687cb044c8a0635f982a4db98
      https://github.com/llvm/llvm-project/commit/c96306db2cad4cf687cb044c8a0635f982a4db98
  Author: AMS21 <AMS21.github at gmail.com>
  Date:   2023-06-18 (Sun, 18 Jun 2023)

  Changed paths:
    M clang-tools-extra/clang-tidy/llvmlibc/InlineFunctionDeclCheck.cpp
    M clang-tools-extra/docs/ReleaseNotes.rst
    M clang-tools-extra/test/clang-tidy/checkers/llvmlibc/inline-function-decl.hpp

  Log Message:
  -----------
  [clang-tidy] Fix `llvmlibc-inline-function-decl` false positives for templated function definitions

For a declaration the `FunctionDecl` begin location does not include the
template parameter lists, but for some reason if you have a separate
definitions to the declaration the begin location does include them.
With this patch we now correctly handle that case.

This fixes llvm#62746

Reviewed By: PiotrZSL

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


  Commit: 82d4dc20efbd72e20b430913e985e38997c7a3e8
      https://github.com/llvm/llvm-project/commit/82d4dc20efbd72e20b430913e985e38997c7a3e8
  Author: AMS21 <AMS21.github at gmail.com>
  Date:   2023-06-18 (Sun, 18 Jun 2023)

  Changed paths:
    M clang-tools-extra/clang-tidy/performance/MoveConstArgCheck.cpp
    M clang-tools-extra/docs/ReleaseNotes.rst
    M clang-tools-extra/test/clang-tidy/checkers/performance/move-const-arg.cpp

  Log Message:
  -----------
  [clang-tidy] Improve `performance-move-const-arg` message when no move constructor is available

We now display a simple note if the reason is that the used class does not
support move semantics.

This fixes llvm#62550

Reviewed By: PiotrZSL

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


Compare: https://github.com/llvm/llvm-project/compare/ed0620fd24a1...82d4dc20efbd