Index: lib/IR/BasicBlock.cpp =================================================================== --- lib/IR/BasicBlock.cpp +++ lib/IR/BasicBlock.cpp @@ -340,9 +340,7 @@ // This is perhaps a little conservative because constructs like // CleanupBlockInst are pretty easy to split. However, SplitBlockPredecessors // cannot handle such things just yet. - if (FirstNonPHI->isEHPad()) - return false; - return true; + return !FirstNonPHI->isEHPad(); } /// This splits a basic block into two at the specified Index: lib/IR/InlineAsm.cpp =================================================================== --- lib/IR/InlineAsm.cpp +++ lib/IR/InlineAsm.cpp @@ -288,8 +288,7 @@ if (!STy || STy->getNumElements() != NumOutputs) return false; break; - } - - if (Ty->getNumParams() != NumInputs) return false; - return true; + } + + return Ty->getNumParams() == NumInputs; } Index: lib/IR/Instructions.cpp =================================================================== --- lib/IR/Instructions.cpp +++ lib/IR/Instructions.cpp @@ -1693,9 +1693,7 @@ bool ExtractElementInst::isValidOperands(const Value *Val, const Value *Index) { - if (!Val->getType()->isVectorTy() || !Index->getType()->isIntegerTy()) - return false; - return true; + return Val->getType()->isVectorTy() && Index->getType()->isIntegerTy(); } Index: lib/IR/Verifier.cpp =================================================================== --- lib/IR/Verifier.cpp +++ lib/IR/Verifier.cpp @@ -1517,12 +1517,9 @@ unsigned LastSlot = Attrs.getNumSlots() - 1; unsigned LastIndex = Attrs.getSlotIndex(LastSlot); - if (LastIndex <= Params - || (LastIndex == AttributeSet::FunctionIndex - && (LastSlot == 0 || Attrs.getSlotIndex(LastSlot - 1) <= Params))) - return true; - - return false; + return LastIndex <= Params || + (LastIndex == AttributeSet::FunctionIndex && + (LastSlot == 0 || Attrs.getSlotIndex(LastSlot - 1) <= Params)); } /// \brief Verify that statepoint intrinsic is well formed.