Refactor: Simplify boolean conditional return statements in clang-modernize

[LegalizeAdulthood]

Summary

Use clang-tidy to simplify boolean conditional return statements

[alexfh]

LG. Do you need me to commit this for you?

clang-modernize/UseNullptr/NullptrActions.cpp

bool allArgUsesValid(const CastExpr *CE) {
     MacroArgUsageVisitor ArgUsageVisitor(SM.getFileLoc(CastLoc), SM);
     if (const Decl *D = ContainingAncestor.get<Decl>())
       ArgUsageVisitor.TraverseDecl(const_cast<Decl *>(D));
     else if (const Stmt *S = ContainingAncestor.get<Stmt>())
       ArgUsageVisitor.TraverseStmt(const_cast<Stmt *>(S));
     else
       llvm_unreachable("Unhandled ContainingAncestor node type");
 
-    if (ArgUsageVisitor.foundInvalid())
-      return false;
-
-    return true;
+    return !ArgUsageVisitor.foundInvalid();
   }
 
   /// \brief Given the SourceLocation for a macro arg expansion, finds the
   /// non-macro SourceLocation of the macro the arg was passed to and the
   /// non-macro SourceLocation of the argument in the arg list to that macro.
   /// These results are returned via \c MacroLoc and \c ArgLoc respectively.
   /// These values are undefined if the return value is false.
   ///
while (1) {
       std::pair<FileID, unsigned> LocInfo = SM.getDecomposedLoc(Loc);
       const SrcMgr::SLocEntry *E = &SM.getSLocEntry(LocInfo.first);
       const SrcMgr::ExpansionInfo &Expansion = E->getExpansion();
 
       Loc = Expansion.getExpansionLocStart();
 
       if (!Expansion.isMacroArgExpansion()) {
         if (Loc.isFileID()) {
-          if (Loc == TestMacroLoc)
-            // Match made.
-            return true;
-          return false;
+          return Loc == TestMacroLoc;
         }
         // Since Loc is still a macro ID and it's not an argument expansion, we
         // don't need to do the work of handling an argument expansion. Simply
         // keep recursively expanding until we hit a FileID or a macro arg
         // expansion or a macro arg expansion.
         continue;
       }