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

    <tr>
        <th>Summary</th>
        <td>
            llvm/lib/Transforms/Scalar/DFAJumpThreading.cpp:548: Poor choice of range variable type ?
        </td>
    </tr>

    <tr>
      <th>Labels</th>
      <td>
            code-quality,
            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/Scalar/DFAJumpThreading.cpp:548:19: performance: Range variable 'Path' should be declared as const reference. [iterateByValue]

Source code is

    for (PathType Path : LoopPaths) {

Suggest code rework.
</pre>
<img width="1px" height="1px" alt="" src="http://email.email.llvm.org/o/eJykkr-O2zAMh5-GXogzbPpfPHhIGngoOhx6h-6UzNjqKZYryVfk7QulBzQ3dzFMECI__D5yCGZeRQZoTtCcM97j4vwwaVWVdabcdBteIkejkVe2tyAe9bbpRfQbBr4FqI5QnKH4-Fr7fgUarVFA46vnNVycvwag8UWzZQ80nsfj1_26vS5eeDLrnOttg-rY1AeojmUP1RE38ekZr1pS-Z3XWfCdvWFlBYG6Z44LUIdhcbudUAlOoi17mZADareGiF4u4mXVkiM0JxPFc5TT7QfbXaA5P1K_uN1rQe0mQRMeO4iIF-cR6JB2vt42wfSDieubc1sqAlCP0J0-jdznWUL8O9PLb-ff8mwaqqmves5kKLuyLruiL9psGajmshDFBbdl0St14YllKqlRNVVV1WZmoILqoqaWqGqKNm8O7eHS911Td3XPiqEu5MrG5klA7vycmRB2GfqCDm1mWYkNd8VEiejp187WxBsQAX0Boru36hgfjFE6Bz-kzpPa5wB1YU2I4d-KaKKV4X-V47NzHvXijBZ0F_SfbccUOVRjtns7LDFu95OjEWicTVx2lWt33_-BkXA3736KjkDjPYXEcg_iTwAAAP__7nrtdA">