[clang-tools-extra] [clang-tidy][NFC] Refactor `modernize-pass-by-value` check code and tests (PR #140753)

[Carlos Galvez via cfe-commits]

carlosgalvezp wrote:

Thanks, I didn't notice they were the same. In that case they should be removed and have nothing. Optionally one can add a human-readable comment like "Should not trigger here" or something, but it's not necessary.

https://github.com/llvm/llvm-project/pull/140753