nit: rename to validateInternalBranches()

Author: Jinjie-Huang

bolt/include/bolt/Core/BinaryFunction.h

@@ -2320,15 +2320,15 @@ class BinaryFunction {
   /// zero-value bytes.
   bool isZeroPaddingAt(uint64_t Offset) const;
 
-  /// Validates if the target of an external direct branch/call is a valid
+  /// Validate if the target of an external direct branch/call is a valid
   /// executable instruction.
   /// Return true if the target is valid, false otherwise.
   bool validateExternalBranch(uint64_t TargetAddress);
 
-  /// Validates if the target of an internal direct branch/call is a valid
+  /// Validate if the target of any internal direct branch/call is a valid
   /// executable instruction.
-  /// Return true if the target is valid, false otherwise.
-  bool validateInternalBranch();
+  /// Return true if all the targets are valid, false otherwise.
+  bool validateInternalBranches();
 
   /// Check that entry points have an associated instruction at their
   /// offsets after disassembly.

bolt/lib/Core/BinaryFunction.cpp

@@ -1947,7 +1947,7 @@ bool BinaryFunction::validateExternalBranch(uint64_t TargetAddress) {
   return true;
 }
 
-bool BinaryFunction::validateInternalBranch() {
+bool BinaryFunction::validateInternalBranches() {
   if (!isSimple())
     return true;
 

bolt/lib/Rewrite/RewriteInstance.cpp

@@ -3498,7 +3498,7 @@ void RewriteInstance::disassembleFunctions() {
     if (!shouldDisassemble(Function))
       continue;
 
-    Function.validateInternalBranch();
+    Function.validateInternalBranches();
     Function.postProcessEntryPoints();
     Function.postProcessJumpTables();
   }