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

    <tr>
        <th>Summary</th>
        <td>
            llvm/utils/TableGen/ARMTargetDefEmitter.cpp:173: Poor choice of range variable type ?
        </td>
    </tr>

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

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

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

<pre>
    Static analyser cppcheck says:

llvm/utils/TableGen/ARMTargetDefEmitter.cpp:173:13: performance: Range variable 'CppSpelling' should be declared as const reference. [iterateByValue]

Source code is

  for (auto CppSpelling : CppSpellings)
    OS << "  &" << CppSpelling << ",\n";



</pre>
<img width="1px" height="1px" alt="" src="http://email.email.llvm.org/o/eJyckk-PnDwMxj-NuVg7ShxYdg4c5s_Le6pa7ax6D8FA2gyhSViJb19lOqvOXnt5wHFi_2Q_OkY7zswNVEeozoVe0-RD05tOybLofL81l6STNahn7bbIAc2ymInNT4x6i6AOIM4g7urc-xWoXZN1Eah9053j_3kGag-vX950GDmdefjvalPisDPLAuoga5U1Cy4cBh-uejacw1c9j4zvOthcCIHq07JcFnbOziNQjXHyq-uxY-zZOB24Rx3R-DkmDDxw4NnwDqE62sRBJz5u37VbGarzI_bFr8EwGt8z2viYQRx8QKAXvSaPD90x8z3EEWj_8QTx6wVBnUCdEIgQgZ7z9370ucrHLaATVKc5_6njI8IfLfpG9Xu11wU3spbPspSyVMXUvJAWVa2EEIb2g-z7qtL1oJ5VuVea9UthGxJUikruZV0R0Y7KQUgpBAkhWNYDlIKv2rpd3t7Oh7GwMa7c7KkWVDjdsYs3fxDlCT39WrWzabszE92Wrg6dnXXYnvLubbIcc746F6HJ-aduHSOUwtmY4t9GySbHzb-7Br95H9BM3hpGP2D4bJi0LYyg2mINrplSWm5-pRaoHW2a1m5nfO58B7iBLsH_YJOA2tsUMtBtEL8DAAD__4c4-VA">