<table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>Issue</th>
<td>
<a href=https://github.com/llvm/llvm-project/issues/89192>89192</a>
</td>
</tr>
<tr>
<th>Summary</th>
<td>
clang/utils/TableGen/MveEmitter.cpp: 2 * pointless copy ?
</td>
</tr>
<tr>
<th>Labels</th>
<td>
good first issue,
clang,
code-cleanup,
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:
1.
clang/utils/TableGen/MveEmitter.cpp:663:51: performance: Function parameter 'IntegerArgs' should be passed by const reference. [passedByValue]
Source code is
IRBuilderResult(StringRef CallPrefix, std::vector<Ptr> Args,
std::set<unsigned> AddressArgs,
std::map<unsigned, std::string> IntegerArgs)
2.
clang/utils/TableGen/MveEmitter.cpp:731:38: performance: Function parameter 'Args' should be passed by const reference. [passedByValue]
Source code is
IRIntrinsicResult(StringRef IntrinsicID, std::vector<const Type *> ParamTypes,
std::vector<Ptr> Args)
</pre>
<img width="1px" height="1px" alt="" src="http://email.email.llvm.org/o/eJy8lE1vpDgQhn-NuVhp0WVo4MChP8KqDytFSbR3Yxe0d43N-iMa_v3IpCfT0fRhZg7DAeSqUtVb7wNw79VoEFtSHkh5yngMF-taKXq2LbLeyqV9CTwoQbnhevHoqJhncUHxH_V88YTtSX4i-fW-3dyehOZmJNDFoLQn0L3yXuNfaAh0f7_h46RCQLcR80zYfrdjhO3LLWF7OqMbrJu4EZiOXTQiKGvozB2fMKCjBKqzCTii27vRE6iov9ioJe2Rztx7lLRfqLDGB-pwQIdG4IaS8vCePSz_cB2RlKdbvS82OoFUWIlU-dsMPT8fotIS3TP6qAOB-iU4ZcZnHOiRa_3kcFBfCBypDzKZwvZvKIJ1hB2fgiPskb4rPV4bfpR5DIQdo1k5yLVQSofef67_4fpoMPH5tsGtBL9qTD0_udXcrga_jaxiiRarfxrZH2N1Nmlxr8QdWh-58-k-rnclr8uMlMA-mfeUlkiBe_zuY75anMmWyYY1PMN2W21ZwaCsq-zSMlYP5a7BeihkUdZlU4i8B1bXu6HqJSsz1UIORV5s67yBpqg2wKoSRFMXecOBQU2KHCeu9Ebrt2lj3Zgp7yO2dbNtINO8R-3XrxpgtFbSQTkf6FpDANIiAFfa305W4oPQyE2cPwf_j1yrsKRgecpcm0Y-9HH0pMi18sF_FxFU0Nj-ymtEIflMZ6tM0Og9FXZeKGFdFp1uLyHM618GOgLdqMIl9hthJwJdmnl9PMzO_osiEOjWDdPM1YivAQAA___Wa4QA">