[all-commits] [llvm/llvm-project] e96f9c: [clang-tidy] Remove IncludeInserter from MoveConst...

[Nathan James via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: e96f9cca3b1b346cf9764c4a3b7417a46ef59f12
      https://github.com/llvm/llvm-project/commit/e96f9cca3b1b346cf9764c4a3b7417a46ef59f12
  Author: Nathan James <n.james93 at hotmail.co.uk>
  Date:   2021-02-23 (Tue, 23 Feb 2021)

  Changed paths:
    M clang-tools-extra/clang-tidy/performance/MoveConstructorInitCheck.cpp
    M clang-tools-extra/clang-tidy/performance/MoveConstructorInitCheck.h
    M clang-tools-extra/docs/clang-tidy/checks/list.rst
    M clang-tools-extra/docs/clang-tidy/checks/performance-move-constructor-init.rst

  Log Message:
  -----------
  [clang-tidy] Remove IncludeInserter from MoveConstructorInit check.

This check registers an IncludeInserter, however the check itself doesn't actually emit any fixes or includes, so the inserter is redundant.

>From what I can tell the fixes were removed in D26453(rL290051) but the inserter was left in, probably an oversight.

Reviewed By: aaron.ballman

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