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

    <tr>
        <th>Summary</th>
        <td>
            clang/include/clang/Interpreter/Interpreter.h:53: Small performance issue ?
        </td>
    </tr>

    <tr>
      <th>Labels</th>
      <td>
            clang,
            code-quality
      </td>
    </tr>

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

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

<pre>
    Static analyser cppcheck says:

clang/include/clang/Interpreter/Interpreter.h:53:36: performance: Function parameter 'TT' should be passed by const reference. [passedByValue]

Source code is

  void SetTargetTriple(std::string TT) { TargetTriple = TT; }

Maybe better code:

  void SetTargetTriple( const std::string & TT ) { TargetTriple = TT; }

</pre>
<img width="1px" height="1px" alt="" src="http://email.email.llvm.org/o/eJykUkGP2ywQ_TXjy2gjDMaODz4kX2TpO_SUqHcME5sWGxfwSv73Fbupml2pUqVeQDOPmfceeipGOy5EHcgzyEuhtjT50Bk9iLIqBm_27ppUshrVotweKaBeVz2R_o5R7RHECdgF2OPUTi0j8N4u2m2GgPe_Ov8vicIaKFH4WB0mECcpQJxEDeKEK4W7D7NaNOWy3xadrF9wVUHNeQCBN7cb8Abj5DdncCBcVYxkcNhR-yUmDHSnQIumA4I8v6Pn_atyG4G8PCu--i1oQu0NoY3PCOKrtwavlG4qjJRuwa6OgB9jMtm2OMUU7DJiFtMiNGd8foggLhkSZ4TmA-UXtQ-EA6VsJhN_-sQ_Ej_MfeYHXuPthn8rojCdMK1oVUFd2ZQNF1Ie62LqTCMbYlV7PLaaD6RrWd2P1CommnYQJRW244xXrGYla6Vg9aGU9Z1JpUWr1J1MCRWjWVl3cO51PvgwFjbGjbq2KpksnBrIxbekcf7IBQf-X668oZcfm3I27bkpL0Xo8pKXYRsjVMzZmOLvtckmR90_pQ2vs3LuOW74JhZB9MUWXDeltL7lm_fA-9GmaRsO2s_A-6zjcb2swX8jnbKMPB6B9-92Xzv-MwAA__-rIQ5Q">