[AMDGPU][NFC] Remove redundant condition
authorAdam Balogh <adam.balogh@ericsson.com>
Thu, 25 Jun 2020 13:41:36 +0000 (15:41 +0200)
committerAdam Balogh <adam.balogh@ericsson.com>
Wed, 1 Jul 2020 07:04:25 +0000 (09:04 +0200)
Condition `LiteralCount` is checked both in an outer and in an inner
`if` statement in `SIInstrInfo::verifyInstruction()`. This patch removes
the redundant inner check.

The issue was found using `clang-tidy` check under review
`misc-redundant-condition`. See https://reviews.llvm.org/D81272.

Differential Revision: https://reviews.llvm.org/D82555

llvm/lib/Target/AMDGPU/SIInstrInfo.cpp

index ffb4073..f6364b7 100644 (file)
@@ -3775,7 +3775,7 @@ bool SIInstrInfo::verifyInstruction(const MachineInstr &MI,
     }
 
     if (isVOP3(MI) && LiteralCount) {
-      if (LiteralCount && !ST.hasVOP3Literal()) {
+      if (!ST.hasVOP3Literal()) {
         ErrInfo = "VOP3 instruction uses literal";
         return false;
       }