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

    <tr>
        <th>Summary</th>
        <td>
            llvm/lib/Target/AMDGPU/GCNHazardRecognizer.cpp:1587: pointless test ?
        </td>
    </tr>

    <tr>
      <th>Labels</th>
      <td>
            new issue
      </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/Target/AMDGPU/GCNHazardRecognizer.cpp:1587:27: style: Expression is always true because 'else if' condition is opposite to previous condition at line 1585. [multiCondition]

Source code is

 if (DefVALUs >= State.ExecPos)
          PreExecPos = std::min(PreExecPos, DefVALUs);
        else if (DefVALUs < State.ExecPos)

</pre>
<img width="1px" height="1px" alt="" src="http://email.email.llvm.org/o/eJykUstu2zAQ_JrVZVGBIvU86ODYUXpoi6Bpeqeolc2WFgUulcT5-kKG07gFeqoOCwEznNlZjGa2-4moheIGil2il3jwoR1Mr7I86f1wah-ijtagnrQ7MQU082wOZH4i6xOD2oDYgbhM556OIDtne5DdNx32FEF2m8-7u_tHkN3d9stH_arD8JWM30_2lUJq5hnUJivqCtRGrgM5nhytP7cvcyBm6ye0jNo96xNjDAthT0YvTAiyIseEdgRZofHTYOOF7ufZs42E0eMc6Mn6ha8YOqKzE2FW1EWKUNwcFxft9g2HYncd7MEvwRAaPxBavkbQjgiy3tH4ffPpkRHULagdrlej9PaFzL1nkM2F_Pu7D3QBcaVzHNZTqs3RTiDrdxTkFt-0Vxl186fSJf1fK2z_4Z8MrRoa1eiE2qwSZa6EaJrk0GZiLGvRF3JsmrGvyegqJ2p0UZZVM6ossa0UMheZrEUtM9WkeZmTGI2QZTFmSlSQCzpq69K1A6kP-8QyL9RWTaXKxOmeHJ9bJuVEz3gGQcq1dKFd33zolz1DLpzlyO8q0UZH7f8WC2dvp-iIGSNxRFBdsgTXHmKczyWWHchub-Nh6VPjz14Xy3W1OfgfZFbL894Msjvn-hUAAP__OucBEA">