[BOLT] Remove redundant dumps in AsmDump

[Amir]

Summary

Dumping jump table and tail call fdata is covered by subsequent iteration over
successors.

[maksfb]

LGTM

bolt/lib/Passes/AsmDump.cpp

case MCCFIInstruction::OpRestoreState:
     return;
 
   default:
     // Emit regular CFI instructions.
     MAP.emitCFIInstruction(*CFIInstr);
   }
 }
 
-void dumpJumpTableFdata(raw_ostream &OS, const BinaryFunction &BF,
-                        const MCInst &Instr, const std::string &BranchLabel) {
-  StringRef FunctionName = BF.getOneName();
-  const JumpTable *JT = BF.getJumpTable(Instr);
-  for (uint32_t i = 0; i < JT->Entries.size(); ++i) {
-    StringRef TargetName = JT->Entries[i]->getName();
-    const uint64_t Mispreds = JT->Counts[i].Mispreds;
-    const uint64_t Count = JT->Counts[i].Count;
-    OS << "# FDATA: 1 " << FunctionName << " #" << BranchLabel << "# "
-       << "1 " << FunctionName << " #" << TargetName << "# " << Mispreds << " "
-       << Count << '\n';
-  }
-}
-
-void dumpTailCallFdata(raw_ostream &OS, const BinaryFunction &BF,
-                       const MCInst &Instr, const std::string &BranchLabel) {
-  const BinaryContext &BC = BF.getBinaryContext();
-  StringRef FunctionName = BF.getOneName();
-  auto CallFreq = BC.MIB->getAnnotationWithDefault<uint64_t>(Instr, "Count");
-  const MCSymbol *Target = BC.MIB->getTargetSymbol(Instr);
-  const BinaryFunction *TargetBF = BC.getFunctionForSymbol(Target);
-  if (!TargetBF)
-    return;
-  OS << "# FDATA: 1 " << FunctionName << " #" << BranchLabel << "# "
-     << "1 " << TargetBF->getPrintName() << " 0 "
-     << "0 " << CallFreq << '\n';
-}
-
 void dumpTargetFunctionStub(raw_ostream &OS, const BinaryContext &BC,
                             const MCSymbol *CalleeSymb,
                             const BinarySection *&LastCS) {
   const BinaryFunction *CalleeFunc = BC.getFunctionForSymbol(CalleeSymb);
   if (!CalleeFunc || CalleeFunc->isPLTFunction())
     return;
 
   if (CalleeFunc->getOriginSection() != LastCS) {
for (const MCInst &Instr : *BB) {
         }
       }
 
       if (&Instr == LastInst && (BB->succ_size() || IsCall))
         OS << BranchLabel << ":\n";
 
       BC.InstPrinter->printInst(&Instr, 0, "", *BC.STI, OS);
       OS << '\n';
-
-      // Dump profile data in FDATA format (as parsed by link_fdata).
-      if (BC.MIB->getJumpTable(Instr))
-        dumpJumpTableFdata(OS, BF, Instr, BranchLabel);
-      else if (BC.MIB->isTailCall(Instr))
-        dumpTailCallFdata(OS, BF, Instr, BranchLabel);
     }
 
     // Dump profile data in FDATA format (as parsed by link_fdata).
     for (const BinaryBasicBlock *Succ : BB->successors()) {
       const BinaryBasicBlock::BinaryBranchInfo BI = BB->getBranchInfo(*Succ);
       if (!BI.MispredictedCount && !BI.Count)
         continue;