<table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>Issue</th>
<td>
<a href=https://github.com/llvm/llvm-project/issues/93003>93003</a>
</td>
</tr>
<tr>
<th>Summary</th>
<td>
llvm/lib/Target/AMDGPU/SIISelLowering.cpp:7596: Pointless code ?
</td>
</tr>
<tr>
<th>Labels</th>
<td>
backend:AMDGPU,
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:
llvm/lib/Target/AMDGPU/SIISelLowering.cpp:7596:31: style: Redundant condition: IsD16. '!IsD16 || (IsD16 && Unpacked)' is equivalent to '!IsD16 || Unpacked' [redundantCondition]
Source code is
int MaskPopDwords = (!IsD16 || (IsD16 && Unpacked)) ?
DMaskPop : (DMaskPop + 1) / 2;
</pre>
<img width="1px" height="1px" alt="" src="http://email.email.llvm.org/o/eJykUs1uozAQfprhMmqExxDDgUMSxKrSVqo22wcwtkO8dW2KTau8_QqabnrYy2oviBmY72fmkzHawRvTQLmHss3knM5harTqOSuyPuhLc0wyWYXSS3eJZkI1jups1DNGeYnAd5C3kF-fzr29AHXO9kDdTzkNJgF1u4f22-MTUHe8vz8a9z28m8n6YaPGEfhOlPUW-I4z4DuM6eLM8vLD6Nlr6ROq4LVNNvilfR9btt0gkABia4EgDiAOCFRda9oCbfHJj1I9Gw1UAwm0Ec3rbN-kMz5hCn-DuI0IhHI_fUo4_FFQtl_dHsM8KYMqaIM2fv2CaH3CBxmfH8PYvodJRwTeLiL_SXeNwLtPSMT2iojLKoCqW017ZOvv1CEB33_MZLrhuua1zEzDBNvybVlVRXZuGKP-lOsTE1JI01eqLvuyYozz2ghd1JltKKciL4nyilV5sRFCVFTyk1SiKvLqBEVuXqR1m-XkmzANmY1xNk3N85xnTvbGxTVURP3ix2vgu88gENABiJbF3b3O0tl0WZplm03NgnfXz0OEInc2pnhjSDY50_xHxvAxWJ-cifHjaMC7bJ5cc05pXKNMHVA32HSe-40KK82VbVE1TuGXUQvb6jUCdavd3wEAAP___fL21Q">