Refactor: Simplify boolean conditional return statements in lib/Target/PowerPC

[LegalizeAdulthood]

Summary

Use clang-tidy to simplify boolean conditional return statements

[craig.topper]

lib/Target/PowerPC/PPCISelLowering.cpp

11204 ↗

(On Diff #26411)

Push the negate through

[wschmidt]

With Craig's suggestion, LGTM.

[LegalizeAdulthood]

lib/Target/PowerPC/PPCISelLowering.cpp

11204 ↗

(On Diff #26411)

Fixed.

[LegalizeAdulthood]

Update from comments, clang-format

[LegalizeAdulthood]

Can someone commit this for me, please? Thanks.

Richard Thomson <[e-mail]>

[LegalizeAdulthood]

Update from latest.
Update from comments.
I do not have commit access.

llvm/trunk/lib/Target/PowerPC/PPCFastISel.cpp

bool PPCFastISel::isLoadTypeLegal(Type *Ty, MVT &VT) {
   return false;
 }
 
 bool PPCFastISel::isValueAvailable(const Value *V) const {
   if (!isa<Instruction>(V))
     return true;
 
   const auto *I = cast<Instruction>(V);
-  if (FuncInfo.MBBMap[I->getParent()] == FuncInfo.MBB)
-    return true;
-
-  return false;
+  return FuncInfo.MBBMap[I->getParent()] == FuncInfo.MBB;
 }
 
 // Given a value Obj, create an Address object Addr that represents its
 // address.  Return false if we can't handle it.
 bool PPCFastISel::PPCComputeAddress(const Value *Obj, Address &Addr) {
   const User *U = nullptr;
   unsigned Opcode = Instruction::UserOp1;
   if (const Instruction *I = dyn_cast<Instruction>(Obj)) {

llvm/trunk/lib/Target/PowerPC/PPCISelDAGToDAG.cpp

for (ValueRotInfo &VRI : ValueRotsVec) {
               EffRLAmt += 32;
           } else {
             return false;
           }
         } else if (VRI.Repl32 != BG.Repl32) {
           return false;
         }
 
-        if (VRI.RLAmt != EffRLAmt)
-          return false;
-
-        return true;
+        return VRI.RLAmt == EffRLAmt;
       };
 
       for (auto &BG : BitGroups) {
         if (!MatchingBG(BG))
           continue;
 
         if (BG.StartIdx <= BG.EndIdx) {
           for (unsigned i = BG.StartIdx; i <= BG.EndIdx; ++i)

llvm/trunk/lib/Target/PowerPC/PPCISelLowering.cpp

 
 /// \brief Returns true if it is beneficial to convert a load of a constant
 /// to just the constant itself.
 bool PPCTargetLowering::shouldConvertConstantLoadToIntImm(const APInt &Imm,
                                                           Type *Ty) const {
   assert(Ty->isIntegerTy());
 
   unsigned BitSize = Ty->getPrimitiveSizeInBits();
-  if (BitSize == 0 || BitSize > 64)
-    return false;
-  return true;
+  return !(BitSize == 0 || BitSize > 64);
 }
 
 bool PPCTargetLowering::isTruncateFree(Type *Ty1, Type *Ty2) const {
   if (!Ty1->isIntegerTy() || !Ty2->isIntegerTy())
     return false;
   unsigned NumBits1 = Ty1->getPrimitiveSizeInBits();
   unsigned NumBits2 = Ty2->getPrimitiveSizeInBits();
   return NumBits1 == 64 && NumBits2 == 32;

llvm/trunk/lib/Target/PowerPC/PPCVSXSwapRemoval.cpp

private:
 
   // Dump a description of the entries in the swap vector.
   void dumpSwapVector();
 
   // Return true iff the given register is in the given class.
   bool isRegInClass(unsigned Reg, const TargetRegisterClass *RC) {
     if (TargetRegisterInfo::isVirtualRegister(Reg))
       return RC->hasSubClassEq(MRI->getRegClass(Reg));
-    if (RC->contains(Reg))
-      return true;
-    return false;
+    return RC->contains(Reg);
   }
 
   // Return true iff the given register is a full vector register.
   bool isVecReg(unsigned Reg) {
     return (isRegInClass(Reg, &PPC::VSRCRegClass) ||
             isRegInClass(Reg, &PPC::VRRCRegClass));
   }