Refactor: Simplify boolean conditional return statements in tools/yaml2obj

[LegalizeAdulthood]

Summary

Use clang-tidy to simplify boolean conditional return statements

[atanasyan]

tools/yaml2obj/yaml2coff.cpp

117

Might be the following code even better?

return parseSections() && parseSymbols();

[silvas]

The way it already is makes it immediately clear that this is forwarding errors, and immediately indicates what the sense of the error bool is. The suggested alternatives require head scratching/digging around. I vote to keep it as-is.

tools/yaml2obj/yaml2coff.cpp

for (std::vector<COFFYAML::Symbol>::iterator i = Obj.Symbols.begin(),
       Sym.Header.Type |= Sym.ComplexType << COFF::SCT_COMPLEX_TYPE_SHIFT;
     }
     return true;
   }
 
   bool parse() {
     if (!parseSections())
       return false;
-    if (!parseSymbols())
-      return false;
-    return true;
+    return parseSymbols();
   }
 
   unsigned getStringIndex(StringRef Str) {
     StringMap<unsigned>::iterator i = StringTableMap.find(Str);
     if (i == StringTableMap.end()) {
       unsigned Index = StringTable.size();
       StringTable.append(Str.begin(), Str.end());
       StringTable.push_back(0);