<table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>Issue</th>
<td>
<a href=https://github.com/llvm/llvm-project/issues/90281>90281</a>
</td>
</tr>
<tr>
<th>Summary</th>
<td>
llvm/lib/Target/AMDGPU/SIMachineFunctionInfo.cpp:113: Possible bad expression ?
</td>
</tr>
<tr>
<th>Labels</th>
<td>
code-quality,
llvm
</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/SIMachineFunctionInfo.cpp:113:38: style: Same expression on both sides of '||'. [duplicateExpression]
Source code is
if (!AMDGPU::isGraphics(CC) ||
((CC == CallingConv::AMDGPU_CS || CC == CallingConv::AMDGPU_CS) &&
ST.hasArchitectedSGPRs())) {
Suggest code rework.
</pre>
<img width="1px" height="1px" alt="" src="http://email.email.llvm.org/o/eJykUs1unDAQfprhYgXhMbvAgQNhS5RDpKik58qYAdw4mNom7b59BYva9Fap1siyZX9_0ie91-NMVMLpHk6XSK5hsq7sVSd4GnW2v5ZtkEErJmdprp4cU8uiJlKvzMurB1FBcoHk2I15fwNsjO4AmxfpRgqATfV0eXj-Ati0j09STXqmZp1V0HZ-nAcbq2UBUXEuQFQiB1ExH66GtkMr34jRz8WR99rOzM6ss2FiXvfkmR0YYAZZvQ1mMYPTfb8uRisZ6NNvFJwuHz22dnWKmLI9Me0_vjCmN8YckB-WRQWi0v7ByWXSygPmdQ1YsEPzQG1rh-V1zUBcQFxYLY3R81jb-f3GcmP8WrcHmP3L310Lz9vctNqXeJK-cmrSgVSgvn14_ux38eI2DLL7v-Ku40g-3PI6-mHdaxz1pegLUciISp7xlJ-L4pxHU9mL00AZ5UpxpTDJ0oGrPpNDN3SJkEMW6RITTJMUz4i8wDwuUErB-akoMMly2UOa0JvUJt6qEFs3Rtr7lcoiwZxHRnZk_F42xM3R3fdVGh2ugAhYA-KtQbh10ZXb5a5bRw9pYrQP_g9r0MFQ-d99Y8_We90ZYp3sPzYNRBOtzpRTCMvecmwAm1GHae1iZXfVQ3wzuTj7jdQmvsf1gM2e-FcAAAD__340BBI">