<table border="1" cellspacing="0" cellpadding="8">
    <tr>
        <th>Issue</th>
        <td>
            <a href=https://github.com/llvm/llvm-project/issues/89201>89201</a>
        </td>
    </tr>

    <tr>
        <th>Summary</th>
        <td>
            llvm/lib/Transforms/Instrumentation/DataFlowSanitizer.cpp:360: pointless copy of a vector ?
        </td>
    </tr>

    <tr>
      <th>Labels</th>
      <td>
            code-quality,
            llvm,
            llvm:transforms
      </td>
    </tr>

    <tr>
      <th>Assignees</th>
      <td>
      </td>
    </tr>

    <tr>
      <th>Reporter</th>
      <td>
          dcb314
      </td>
    </tr>
</table>

<pre>
    Static analyser cppcheck says:

llvm/lib/Transforms/Instrumentation/DataFlowSanitizer.cpp:360:45: performance: Function parameter 'ArgumentIndexMapping' should be passed by const reference. [passedByValue]

Source code is

 TransformedFunction(FunctionType *OriginalType, FunctionType *TransformedType,
                      std::vector<unsigned> ArgumentIndexMapping)

</pre>
<img width="1px" height="1px" alt="" src="http://email.email.llvm.org/o/eJysUsFu5CgQ_ZryBaWFCzttH3xw0msph9UeEu0d47KbXRoYwJnxfP0Id2emNcpxOCBKDx6vXj0Zo14sUQf1E9SnQq7p7EI3qVGUVTG6aetek0xaMWml2SIFprxXZ1L_syi3CKIHfgJ-2415vwAORo-Aw1uQNs4uXCLg8GJjCuuFbGZzFnA4ySQH476-SquT_k7hoLwH0YtHDqKvahA98xQygbSKcjmsVuXXzMsgL5QoMMBjH5ad-MVO9O1v6b22C-CRxbNbzcRGYl7GSBMbN6acjYkFmimQVXRgUD9d0aftX2lWgvp039CrW4MiptxETMd7hP1sj6YPXYDNx_Ft88QA-3-CXrSVJteAz-x3_I7lduVG_-mKacqOi_6dVHIBxPNq9wFOIP5inxvRXgmLqRNTK1pZUFceS1GJ-sh5ce7aum2Jj_VY8vko1dzwpkbV1tTg41w3U6E75FjxqmxKjmXFD1zVkhqJpGiq57KBitNFanPI4z-4sBQ6xpW6pkVeFkaOZOIeMMRs5MOXVRqdNkDM7SJeU3NfiD7dhQdzMkOXkYdxXSJU3OiY4q__kk6Guj-XPuadtslQjEw5vzE3M8munjMQQ7EG051T8nv-cQAcFp3O63hQbldwE5IF--D-I5UAh92UrGb35UcAAAD__4zqIJU">