llvm/include/llvm/Frontend/OpenMP/OMPIRBuilder.h:201: Pointless copy of object ?

[dcb314]

Static analyser cppcheck says:

llvm/include/llvm/Frontend/OpenMP/OMPIRBuilder.h:201:46: performance: Function parameter 'RHS' should be passed by const reference. [passedByValue]

Source code is

bool operator<(const TargetRegionEntryInfo RHS) const {

Suggest new code:

bool operator<(const TargetRegionEntryInfo & RHS) const {

[llvmbot]

@llvm/issue-subscribers-openmp

Author: None (dcb314)

Static analyser cppcheck says:

llvm/include/llvm/Frontend/OpenMP/OMPIRBuilder.h:201:46: performance: Function parameter 'RHS' should be passed by const reference. [passedByValue]

Source code is

bool operator<(const TargetRegionEntryInfo RHS) const {

Suggest new code:

bool operator<(const TargetRegionEntryInfo & RHS) const {

[shiltian]

feel free to submit a PR for this simple fix