[all-commits] [llvm/llvm-project] 556e4d: [clang-tidy] Fix performance-move-const-arg false ...

David Rivera via All-commits all-commits at lists.llvm.org
Thu Feb 27 03:35:33 PST 2025


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 556e4dbdcdfc88bc52b43324c4b3af0100c75cc4
      https://github.com/llvm/llvm-project/commit/556e4dbdcdfc88bc52b43324c4b3af0100c75cc4
  Author: David Rivera <110955221+RiverDave at users.noreply.github.com>
  Date:   2025-02-27 (Thu, 27 Feb 2025)

  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] Fix performance-move-const-arg false negative in ternary… (#128402)

This PR aims to fix `performance-move-const-arg` #126515

## Changes
Enhanced the `performance-move-arg` check in Clang-Tidy to detect cases
where `std::move` is used
in **ternary operator expressions which was not being matched therefore
being tagged as a false negative**

## Testing
- A new mock class has been where the changes have been tested & all
tests pass

I'd appreciate any feedback since this is my first time contributing to
LLVM.



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


More information about the All-commits mailing list