Index: lib/Transforms/Utils/Local.cpp =================================================================== --- lib/Transforms/Utils/Local.cpp +++ lib/Transforms/Utils/Local.cpp @@ -288,14 +288,10 @@ // We don't want debug info removed by anything this general, unless // debug info is empty. if (DbgDeclareInst *DDI = dyn_cast(I)) { - if (DDI->getAddress()) - return false; - return true; + return !DDI->getAddress(); } if (DbgValueInst *DVI = dyn_cast(I)) { - if (DVI->getValue()) - return false; - return true; + return !DVI->getValue(); } if (!I->mayHaveSideEffects()) return true; Index: lib/Transforms/Utils/SimplifyCFG.cpp =================================================================== --- lib/Transforms/Utils/SimplifyCFG.cpp +++ lib/Transforms/Utils/SimplifyCFG.cpp @@ -3554,11 +3554,8 @@ // is unreachable. DefaultResult = DefaultResults.size() == 1 ? DefaultResults.begin()->second : nullptr; - if ((!DefaultResult && - !isa(DefaultDest->getFirstNonPHIOrDbg()))) - return false; - - return true; + return DefaultResult || + isa(DefaultDest->getFirstNonPHIOrDbg()); } // ConvertTwoCaseSwitch - Helper function that checks if it is possible to Index: lib/Transforms/Utils/SimplifyIndVar.cpp =================================================================== --- lib/Transforms/Utils/SimplifyIndVar.cpp +++ lib/Transforms/Utils/SimplifyIndVar.cpp @@ -423,10 +423,7 @@ // Only consider affine recurrences. const SCEVAddRecExpr *AR = dyn_cast(S); - if (AR && AR->getLoop() == L) - return true; - - return false; + return AR && AR->getLoop() == L; } /// Iteratively perform simplification on a worklist of users