[all-commits] [llvm/llvm-project] 233d12: [clang-tidy][NFC] Do less unnecessary work in `mod...

[Victor Chernyakin via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 233d1220f8c88b2b0ac8f281f8e63f15216ede38
      https://github.com/llvm/llvm-project/commit/233d1220f8c88b2b0ac8f281f8e63f15216ede38
  Author: Victor Chernyakin <chernyakin.victor.j at outlook.com>
  Date:   2025-10-08 (Wed, 08 Oct 2025)

  Changed paths:
    M clang-tools-extra/clang-tidy/modernize/DeprecatedHeadersCheck.cpp
    M clang-tools-extra/clang-tidy/modernize/DeprecatedHeadersCheck.h

  Log Message:
  -----------
  [clang-tidy][NFC] Do less unnecessary work in `modernize-deprecated-headers` (#160967)

- `IncludeModernizePPCallbacks` creates temporary vectors when all it
needs is constant arrays
- The header replacement strings are `std::string` when they can just be
`StringRef`
- `IncludeModernizePPCallbacks`'s constructor
  1. Takes `LangOptions` by value (the thing is 832 bytes)
2. Stores it, but none of `IncludeModernizePPCallbacks`'s member
functions use it



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications