minor nits

Author: Alexander Yermolovich

bolt/include/bolt/Core/BinaryFunction.h

@@ -428,7 +428,7 @@ class BinaryFunction {
   /// Function order for streaming into the destination binary.
   uint32_t Index{-1U};
 
-  /// Indicates if the function is safe to fold.
+  /// Indicates function is referenced by none-control flow instruction.
   bool HasAddressTaken{false};
 
   /// Get basic block index assuming it belongs to this function.
@@ -820,10 +820,10 @@ class BinaryFunction {
     return nullptr;
   }
 
-  /// Indicates if the function is safe to fold.
+  /// Returns true if function is referenced in none-control flow instructions..
   bool hasAddressTaken() const { return HasAddressTaken; }
 
-  /// Sets the function is not safe to fold.
+  /// Sets whether function is referenced in none-control flow instructions.
   void setHasAddressTaken(bool Hat) { HasAddressTaken = Hat; }
 
   /// Returns the raw binary encoding of this function.

bolt/lib/Passes/IdenticalCodeFolding.cpp

@@ -357,7 +357,7 @@ namespace bolt {
 /// Scans symbol table and creates a bit vector of memory addresses of vtables.
 static void processSymbolTable(const BinaryContext &BC,
                                llvm::BitVector &BitVector) {
-  for (auto [Address, Data] : BC.getBinaryData()) {
+  for (auto &[Address, Data] : BC.getBinaryData()) {
     // Filter out all symbols that are not vtables.
     if (!Data->getName().starts_with("_ZTV"))
       continue;