[all-commits] [llvm/llvm-project] db335d: [clang-tidy] Ignore cxxRewrittenBinaryOperator in ...

[jensmassberg via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: db335d02a5e7d98c24f7006e42129856ba1cd695
      https://github.com/llvm/llvm-project/commit/db335d02a5e7d98c24f7006e42129856ba1cd695
  Author: Jens Massberg <massberg at google.com>
  Date:   2022-11-25 (Fri, 25 Nov 2022)

  Changed paths:
    M clang-tools-extra/clang-tidy/modernize/UseNullptrCheck.cpp
    M clang-tools-extra/test/clang-tidy/checkers/modernize/use-nullptr-cxx20.cpp

  Log Message:
  -----------
  [clang-tidy] Ignore cxxRewrittenBinaryOperator in defaulted function decls in modernize-use-nullptr

The check has produced false positives when checking the default implementation of the spaceship operator.
The default implementation should be skipped by the check.

Modified the existing test so that the check runs into the bug without this fix and add another test case.

Fixes #53961.

Patch by Jens Massberg.

Reviewed By: ilya-biryukov

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