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

    <tr>
        <th>Summary</th>
        <td>
            Dead code in MLRegAllocEvictAdvisor.cpp
        </td>
    </tr>

    <tr>
      <th>Labels</th>
      <td>
            mlgo
      </td>
    </tr>

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

    <tr>
      <th>Reporter</th>
      <td>
          abhishek-kaushik22
      </td>
    </tr>
</table>

<pre>
    In [MLRegAllocEvictAdvisor.cpp](https://github.com/llvm/llvm-project/blob/main/llvm/lib/CodeGen/MLRegAllocEvictAdvisor.cpp#L862-L874), the condition `if (CandidatePos == CandidateVirtRegPos)` is checked twice but the first time it's true, the function returns making the second condition check unnecessary.

@boomanaiden154 can you please take a look because this was introduced in 00f692b94f9aa08ede4aaba6f2aafe17857599c4
</pre>
<img width="1" height="1" alt="" src="http://email.email.llvm.org/o/eJyUklGL3C4UxT-NeZEdjNFJfMjD7M5__hS2sOxD3696k9yOo0PUWfbblwzddl9aKAjCUc45en-QM80RcWT6keljA7UsaR3BLpQXPD-coeaFzlI2Nvn38UvkTD9-fX7F-RBCcv_dyJWDv1FO685dr0wfmRyWUq6ZdQcmT0yeZipLtTuXLkyeQrh9bA_XNX1HV5g82ZAsk6cLUPx0hzbtKXn8Hzf5L6myex728uF56BWThsknXhbkLkVPhVLkbC9o4kwOTxA9eSj4kjJn3ZF1R_5L-0ZrecX5JeXNZC84Ze4WdGf0vLyRQ25ruTtPtObCC12QU2Gyz7ysFT9ypxrdPXbFUteY-QXOFOf7Wcat1adq9wBeY0SHOcP6vmPisC0lbEoXiEAeY6sVdxD5e6r8GhAy8gJn5MBDSmdu0UHdtIUyf4PMKZY1-erQc4pciGlvpDVqMgBiQI8KwMJ-kgATtv2ge22MU40fO286Aw2Oba_aVvd7I5pl1HrSPXaTMb0F7Q146wY34QSqtbJTDY1SSC06ocVeKy12ajKtcd2ketkJN3VMCbwAhd023F1a54Zyrji20gyqbQJYDPkOoZSXMCcm5YbjOt5JsXXOTIlAueTfDoVKwPGIsP2nx-2lf2akqWsY_xnMe8nM5Olnz9sofwQAAP__alQMYQ">