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

    <tr>
        <th>Summary</th>
        <td>
            clang/lib/Driver/ToolChains/Clang.cpp:8122: Poor choice of range variable type ?
        </td>
    </tr>

    <tr>
      <th>Labels</th>
      <td>
            clang:driver,
            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/lib/Driver/ToolChains/Clang.cpp:8122:13: performance: Range variable 'EHVal' should be declared as const reference. [iterateByValue]

Source code is

  for (auto EHVal : EHArgs) {

Suggest code rework.
</pre>
<img width="1px" height="1px" alt="" src="http://email.email.llvm.org/o/eJycksFupDgQhp-muJSCoIDQHDh0p4NyXG1WuZftArxxY8Y2Peq3H9HJaJLrXGy5bH1_lT5zjHZaRHpoTtCcM97S7ENvtKrKOlPe3PrXxMlq5IXdLUpAva56Fv2OkW8RqiMUZyg-V-14mYAGZxXQcA72KgFo-M979zSzXSLQ8LS_yfW6QnU8lERQHcsKqiOuEkYfLrxo2Y__8jIJXjlYVk4QqH1-eWMH1GKc_eYMKkEj2nEQgxxR-yUmDDJKkEVLjtCcbJLASU63N3abQHP-2u2r34IW1N4I2vj1BnH0AYEOvCWP91zce3p-OYYpAnUI7ekbapsmiemDFeSnD-95ZvrKdFXHmfRlW9ZlW1DTZXPfqVJXhRoPj1yUbXswZTs2qh0bLg5dVZrM9lRQXdT0SER12eWtUo3hgnWnSNpGoC7kwtblzl0vuQ9TZmPcpO8KOlDmWImLd6VEH0qqo_mUQUBPe9kbefixsbPpthebcxb6nfagtilCXTgbU_zDTzY56f_SL_7jfUA9e6sF_Yjhu9p0WwWhGrItuH5Oab1_KxqAhsmmeVO59pc91V1_bw9r8P-LTkDDffI9-D78rwAAAP__TWDhyQ">