Apply suggestions from code review

Co-authored-by: Ellis Hoag <[email protected]>

Author: mingmingl-llvm

llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp

@@ -2896,10 +2896,11 @@ void AsmPrinter::emitJumpTableInfo() {
   int NextHotJumpTableIndex = 0, NextColdJumpTableIndex = JT.size() - 1;
   JumpTableIndices.resize(JT.size());
   for (unsigned JTI = 0, JTSize = JT.size(); JTI < JTSize; ++JTI) {
-    if (JT[JTI].Hotness == MachineFunctionDataHotness::Cold)
+    if (JT[JTI].Hotness == MachineFunctionDataHotness::Cold) {
       JumpTableIndices[NextColdJumpTableIndex--] = JTI;
-    else
+    } else {
       JumpTableIndices[NextHotJumpTableIndex++] = JTI;
+    }
   }
 
   if (NextHotJumpTableIndex != 0) {
@@ -2945,7 +2946,7 @@ void AsmPrinter::emitJumpTables(ArrayRef<unsigned> JumpTableIndices,
 
   const auto &JT = MJTI.getJumpTables();
   for (unsigned Index = 0, e = JumpTableIndices.size(); Index != e; ++Index) {
-    const std::vector<MachineBasicBlock *> &JTBBs =
+    ArrayRef<MachineBasicBlock *> JTBBs =
         JT[JumpTableIndices[Index]].MBBs;
 
     // If this jump table was deleted, ignore it.

llvm/lib/CodeGen/TargetLoweringObjectFileImpl.cpp

@@ -669,10 +669,11 @@ getELFSectionNameForGlobal(const GlobalObject *GO, SectionKind Kind,
     // Jump table hotness takes precedence over its enclosing function's hotness
     // if both are available.
     if (JTE) {
-      if (JTE->Hotness == MachineFunctionDataHotness::Hot)
+      if (JTE->Hotness == MachineFunctionDataHotness::Hot) {
         raw_svector_ostream(Name) << ".hot";
-      else if (JTE->Hotness == MachineFunctionDataHotness::Cold)
+      } else if (JTE->Hotness == MachineFunctionDataHotness::Cold) {
         raw_svector_ostream(Name) << ".unlikely";
+      }
     } else if (std::optional<StringRef> Prefix = F->getSectionPrefix()) {
       raw_svector_ostream(Name) << '.' << *Prefix;
       HasPrefix = true;