12f09f445SMaksim Panchenko //===- bolt/Rewrite/BoltDiff.cpp ------------------------------------------===// 2a34c753fSRafael Auler // 3a34c753fSRafael Auler // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. 4a34c753fSRafael Auler // See https://llvm.org/LICENSE.txt for license information. 5a34c753fSRafael Auler // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception 6a34c753fSRafael Auler // 7a34c753fSRafael Auler //===----------------------------------------------------------------------===// 8a34c753fSRafael Auler // 9a34c753fSRafael Auler // RewriteInstance methods related to comparing one instance to another, used 10a34c753fSRafael Auler // by the boltdiff tool to print a report. 11a34c753fSRafael Auler // 12a34c753fSRafael Auler //===----------------------------------------------------------------------===// 13a34c753fSRafael Auler 14a34c753fSRafael Auler #include "bolt/Passes/IdenticalCodeFolding.h" 15a34c753fSRafael Auler #include "bolt/Profile/ProfileReaderBase.h" 16a34c753fSRafael Auler #include "bolt/Rewrite/RewriteInstance.h" 17dbb7316eSAmir Ayupov #include "bolt/Utils/Utils.h" 18f40d25ddSAmir Ayupov #include "llvm/ADT/STLExtras.h" 19a34c753fSRafael Auler #include "llvm/Support/CommandLine.h" 20a34c753fSRafael Auler 21a34c753fSRafael Auler #undef DEBUG_TYPE 22a34c753fSRafael Auler #define DEBUG_TYPE "boltdiff" 23a34c753fSRafael Auler 24a34c753fSRafael Auler using namespace llvm; 25a34c753fSRafael Auler using namespace object; 26a34c753fSRafael Auler using namespace bolt; 27a34c753fSRafael Auler 28a34c753fSRafael Auler namespace opts { 29a34c753fSRafael Auler extern cl::OptionCategory BoltDiffCategory; 30a34c753fSRafael Auler extern cl::opt<bool> NeverPrint; 31*3c357a49SAlexander Yermolovich extern cl::opt<bolt::IdenticalCodeFolding::ICFLevel, false, 32*3c357a49SAlexander Yermolovich llvm::bolt::DeprecatedICFNumericOptionParser> 33*3c357a49SAlexander Yermolovich ICF; 34a34c753fSRafael Auler 35b92436efSFangrui Song static cl::opt<bool> IgnoreLTOSuffix( 36b92436efSFangrui Song "ignore-lto-suffix", 37a34c753fSRafael Auler cl::desc("ignore lto_priv or const suffixes when matching functions"), 38b92436efSFangrui Song cl::init(true), cl::cat(BoltDiffCategory)); 39a34c753fSRafael Auler 40b92436efSFangrui Song static cl::opt<bool> PrintUnmapped( 41b92436efSFangrui Song "print-unmapped", 42a34c753fSRafael Auler cl::desc("print functions of binary 2 that were not matched to any " 43a34c753fSRafael Auler "function in binary 1"), 44a34c753fSRafael Auler cl::cat(BoltDiffCategory)); 45a34c753fSRafael Auler 46b92436efSFangrui Song static cl::opt<bool> PrintProfiledUnmapped( 47b92436efSFangrui Song "print-profiled-unmapped", 48a34c753fSRafael Auler cl::desc("print functions that have profile in binary 1 but do not " 49a34c753fSRafael Auler "in binary 2"), 50a34c753fSRafael Auler cl::cat(BoltDiffCategory)); 51a34c753fSRafael Auler 52b92436efSFangrui Song static cl::opt<bool> PrintDiffCFG( 53b92436efSFangrui Song "print-diff-cfg", 54a34c753fSRafael Auler cl::desc("print the CFG of important functions that changed in " 55a34c753fSRafael Auler "binary 2"), 56a34c753fSRafael Auler cl::cat(BoltDiffCategory)); 57a34c753fSRafael Auler 58a34c753fSRafael Auler static cl::opt<bool> 59a34c753fSRafael Auler PrintDiffBBs("print-diff-bbs", 60a34c753fSRafael Auler cl::desc("print the basic blocks showed in top differences"), 61a34c753fSRafael Auler cl::cat(BoltDiffCategory)); 62a34c753fSRafael Auler 63b92436efSFangrui Song static cl::opt<bool> MatchByHash( 64b92436efSFangrui Song "match-by-hash", 65a34c753fSRafael Auler cl::desc("match functions in binary 2 to binary 1 if they have the same " 66a34c753fSRafael Auler "hash of a function in binary 1"), 67a34c753fSRafael Auler cl::cat(BoltDiffCategory)); 68a34c753fSRafael Auler 69b92436efSFangrui Song static cl::opt<bool> IgnoreUnchanged( 70b92436efSFangrui Song "ignore-unchanged", 71a34c753fSRafael Auler cl::desc("do not diff functions whose contents have not been changed from " 72a34c753fSRafael Auler "one binary to another"), 73a34c753fSRafael Auler cl::cat(BoltDiffCategory)); 74a34c753fSRafael Auler 75b92436efSFangrui Song static cl::opt<unsigned> DisplayCount( 76b92436efSFangrui Song "display-count", 77a34c753fSRafael Auler cl::desc("number of functions to display when printing the top largest " 78a34c753fSRafael Auler "differences in function activity"), 79b92436efSFangrui Song cl::init(10), cl::cat(BoltDiffCategory)); 80a34c753fSRafael Auler 81b92436efSFangrui Song static cl::opt<bool> NormalizeByBin1( 82b92436efSFangrui Song "normalize-by-bin1", 83a34c753fSRafael Auler cl::desc("show execution count of functions in binary 2 as a ratio of the " 84a34c753fSRafael Auler "total samples in binary 1 - make sure both profiles have equal " 85a34c753fSRafael Auler "collection time and sampling rate for this to make sense"), 86a34c753fSRafael Auler cl::cat(BoltDiffCategory)); 87a34c753fSRafael Auler 88c03e6511SAmir Ayupov static cl::opt<bool> 89c03e6511SAmir Ayupov SkipNonSimple("skip-non-simple", 90c03e6511SAmir Ayupov cl::desc("skip non-simple functions in reporting"), 91c03e6511SAmir Ayupov cl::ReallyHidden, cl::cat(BoltDiffCategory)); 92c03e6511SAmir Ayupov 93a34c753fSRafael Auler } // end namespace opts 94a34c753fSRafael Auler 95a34c753fSRafael Auler namespace llvm { 96a34c753fSRafael Auler namespace bolt { 97a34c753fSRafael Auler 98a34c753fSRafael Auler namespace { 99a34c753fSRafael Auler 100a34c753fSRafael Auler /// Helper used to print colored numbers 101a34c753fSRafael Auler void printColoredPercentage(double Perc) { 102a34c753fSRafael Auler if (outs().has_colors() && Perc > 0.0) 103a34c753fSRafael Auler outs().changeColor(raw_ostream::RED); 104a34c753fSRafael Auler else if (outs().has_colors() && Perc < 0.0) 105a34c753fSRafael Auler outs().changeColor(raw_ostream::GREEN); 106a34c753fSRafael Auler else if (outs().has_colors()) 107a34c753fSRafael Auler outs().changeColor(raw_ostream::YELLOW); 108a34c753fSRafael Auler outs() << format("%.2f", Perc) << "%"; 109a34c753fSRafael Auler if (outs().has_colors()) 110a34c753fSRafael Auler outs().resetColor(); 111a34c753fSRafael Auler } 112a34c753fSRafael Auler 113a34c753fSRafael Auler void setLightColor() { 114a34c753fSRafael Auler if (opts::PrintDiffBBs && outs().has_colors()) 115a34c753fSRafael Auler outs().changeColor(raw_ostream::CYAN); 116a34c753fSRafael Auler } 117a34c753fSRafael Auler 118a34c753fSRafael Auler void setTitleColor() { 119a34c753fSRafael Auler if (outs().has_colors()) 120a34c753fSRafael Auler outs().changeColor(raw_ostream::WHITE, /*Bold=*/true); 121a34c753fSRafael Auler } 122a34c753fSRafael Auler 123a34c753fSRafael Auler void setRegularColor() { 124a34c753fSRafael Auler if (outs().has_colors()) 125a34c753fSRafael Auler outs().resetColor(); 126a34c753fSRafael Auler } 127a34c753fSRafael Auler 128a34c753fSRafael Auler } // end anonymous namespace 129a34c753fSRafael Auler 130a34c753fSRafael Auler /// Perform the comparison between two binaries with profiling information 131a34c753fSRafael Auler class RewriteInstanceDiff { 132a34c753fSRafael Auler typedef std::tuple<const BinaryBasicBlock *, const BinaryBasicBlock *, double> 133a34c753fSRafael Auler EdgeTy; 134a34c753fSRafael Auler 135a34c753fSRafael Auler RewriteInstance &RI1; 136a34c753fSRafael Auler RewriteInstance &RI2; 137a34c753fSRafael Auler 138a34c753fSRafael Auler // The map of functions keyed by functions in binary 2, providing its 139a34c753fSRafael Auler // corresponding function in binary 1 140a34c753fSRafael Auler std::map<const BinaryFunction *, const BinaryFunction *> FuncMap; 141a34c753fSRafael Auler 142a34c753fSRafael Auler // The map of basic blocks correspondence, analogue to FuncMap for BBs, 143a34c753fSRafael Auler // sorted by score difference 144a34c753fSRafael Auler std::map<const BinaryBasicBlock *, const BinaryBasicBlock *> BBMap; 145a34c753fSRafael Auler 146a34c753fSRafael Auler // The map of edge correspondence 147a34c753fSRafael Auler std::map<double, std::pair<EdgeTy, EdgeTy>> EdgeMap; 148a34c753fSRafael Auler 149a34c753fSRafael Auler // Maps all known basic blocks back to their parent function 150a34c753fSRafael Auler std::map<const BinaryBasicBlock *, const BinaryFunction *> BBToFuncMap; 151a34c753fSRafael Auler 152a34c753fSRafael Auler // Accounting which functions were matched 153a34c753fSRafael Auler std::set<const BinaryFunction *> Bin1MappedFuncs; 154a34c753fSRafael Auler std::set<const BinaryFunction *> Bin2MappedFuncs; 155a34c753fSRafael Auler 156a34c753fSRafael Auler // Structures for our 3 matching strategies: by name, by hash and by lto name, 157a34c753fSRafael Auler // from the strongest to the weakest bind between two functions 158a34c753fSRafael Auler StringMap<const BinaryFunction *> NameLookup; 159a34c753fSRafael Auler DenseMap<size_t, const BinaryFunction *> HashLookup; 160a34c753fSRafael Auler StringMap<const BinaryFunction *> LTONameLookup1; 161a34c753fSRafael Auler StringMap<const BinaryFunction *> LTONameLookup2; 162a34c753fSRafael Auler 163a34c753fSRafael Auler // Score maps used to order and find hottest functions 164a34c753fSRafael Auler std::multimap<double, const BinaryFunction *> LargestBin1; 165a34c753fSRafael Auler std::multimap<double, const BinaryFunction *> LargestBin2; 166a34c753fSRafael Auler 167a34c753fSRafael Auler // Map multiple functions in the same LTO bucket to a single parent function 168a34c753fSRafael Auler // representing all functions sharing the same prefix 169a34c753fSRafael Auler std::map<const BinaryFunction *, const BinaryFunction *> LTOMap1; 170a34c753fSRafael Auler std::map<const BinaryFunction *, const BinaryFunction *> LTOMap2; 171a34c753fSRafael Auler std::map<const BinaryFunction *, double> LTOAggregatedScore1; 172a34c753fSRafael Auler std::map<const BinaryFunction *, double> LTOAggregatedScore2; 173a34c753fSRafael Auler 174a34c753fSRafael Auler // Map scores in bin2 and 1 keyed by a binary 2 function - post-matching 175a34c753fSRafael Auler DenseMap<const BinaryFunction *, std::pair<double, double>> ScoreMap; 176a34c753fSRafael Auler 177a34c753fSRafael Auler double getNormalizedScore(const BinaryFunction &Function, 178a34c753fSRafael Auler const RewriteInstance &Ctx) { 179a34c753fSRafael Auler if (!opts::NormalizeByBin1) 18040c2e0faSMaksim Panchenko return static_cast<double>(Function.getFunctionScore()) / 18140c2e0faSMaksim Panchenko Ctx.getTotalScore(); 18240c2e0faSMaksim Panchenko return static_cast<double>(Function.getFunctionScore()) / 18340c2e0faSMaksim Panchenko RI1.getTotalScore(); 184a34c753fSRafael Auler } 185a34c753fSRafael Auler 186a34c753fSRafael Auler double getNormalizedScore(const BinaryBasicBlock &BB, 187a34c753fSRafael Auler const RewriteInstance &Ctx) { 188a34c753fSRafael Auler if (!opts::NormalizeByBin1) 18940c2e0faSMaksim Panchenko return static_cast<double>(BB.getKnownExecutionCount()) / 19040c2e0faSMaksim Panchenko Ctx.getTotalScore(); 19140c2e0faSMaksim Panchenko return static_cast<double>(BB.getKnownExecutionCount()) / 19240c2e0faSMaksim Panchenko RI1.getTotalScore(); 193a34c753fSRafael Auler } 194a34c753fSRafael Auler 195d5c03defSFabian Parzefall double getNormalizedScore(BinaryBasicBlock::const_branch_info_iterator BIIter, 196a34c753fSRafael Auler const RewriteInstance &Ctx) { 197a34c753fSRafael Auler double Score = 19840c2e0faSMaksim Panchenko BIIter->Count == BinaryBasicBlock::COUNT_NO_PROFILE ? 0 : BIIter->Count; 199a34c753fSRafael Auler if (!opts::NormalizeByBin1) 200a34c753fSRafael Auler return Score / Ctx.getTotalScore(); 201a34c753fSRafael Auler return Score / RI1.getTotalScore(); 202a34c753fSRafael Auler } 203a34c753fSRafael Auler 204a34c753fSRafael Auler /// Initialize data structures used for function lookup in binary 1, used 205a34c753fSRafael Auler /// later when matching functions in binary 2 to corresponding functions 206a34c753fSRafael Auler /// in binary 1 207a34c753fSRafael Auler void buildLookupMaps() { 208a34c753fSRafael Auler for (const auto &BFI : RI1.BC->getBinaryFunctions()) { 209a34c753fSRafael Auler StringRef LTOName; 210a34c753fSRafael Auler const BinaryFunction &Function = BFI.second; 211a34c753fSRafael Auler const double Score = getNormalizedScore(Function, RI1); 212a34c753fSRafael Auler LargestBin1.insert(std::make_pair<>(Score, &Function)); 213253b8f0aSAmir Ayupov for (const StringRef &Name : Function.getNames()) { 21415d1e517SAmir Ayupov if (std::optional<StringRef> OptionalLTOName = getLTOCommonName(Name)) 215a34c753fSRafael Auler LTOName = *OptionalLTOName; 216a34c753fSRafael Auler NameLookup[Name] = &Function; 217a34c753fSRafael Auler } 218a34c753fSRafael Auler if (opts::MatchByHash && Function.hasCFG()) 219a34c753fSRafael Auler HashLookup[Function.computeHash(/*UseDFS=*/true)] = &Function; 220a34c753fSRafael Auler if (opts::IgnoreLTOSuffix && !LTOName.empty()) { 221a34c753fSRafael Auler if (!LTONameLookup1.count(LTOName)) 222a34c753fSRafael Auler LTONameLookup1[LTOName] = &Function; 223a34c753fSRafael Auler LTOMap1[&Function] = LTONameLookup1[LTOName]; 224a34c753fSRafael Auler } 225a34c753fSRafael Auler } 226a34c753fSRafael Auler 227a34c753fSRafael Auler // Compute LTONameLookup2 and LargestBin2 228a34c753fSRafael Auler for (const auto &BFI : RI2.BC->getBinaryFunctions()) { 229a34c753fSRafael Auler StringRef LTOName; 230a34c753fSRafael Auler const BinaryFunction &Function = BFI.second; 231a34c753fSRafael Auler const double Score = getNormalizedScore(Function, RI2); 232a34c753fSRafael Auler LargestBin2.insert(std::make_pair<>(Score, &Function)); 233253b8f0aSAmir Ayupov for (const StringRef &Name : Function.getNames()) { 23415d1e517SAmir Ayupov if (std::optional<StringRef> OptionalLTOName = getLTOCommonName(Name)) 235a34c753fSRafael Auler LTOName = *OptionalLTOName; 236a34c753fSRafael Auler } 237a34c753fSRafael Auler if (opts::IgnoreLTOSuffix && !LTOName.empty()) { 238a34c753fSRafael Auler if (!LTONameLookup2.count(LTOName)) 239a34c753fSRafael Auler LTONameLookup2[LTOName] = &Function; 240a34c753fSRafael Auler LTOMap2[&Function] = LTONameLookup2[LTOName]; 241a34c753fSRafael Auler } 242a34c753fSRafael Auler } 243a34c753fSRafael Auler } 244a34c753fSRafael Auler 245a34c753fSRafael Auler /// Match functions in binary 2 with functions in binary 1 246a34c753fSRafael Auler void matchFunctions() { 247a34c753fSRafael Auler outs() << "BOLT-DIFF: Mapping functions in Binary2 to Binary1\n"; 248a34c753fSRafael Auler uint64_t BothHaveProfile = 0ull; 249a34c753fSRafael Auler std::set<const BinaryFunction *> Bin1ProfiledMapped; 250a34c753fSRafael Auler 251a34c753fSRafael Auler for (const auto &BFI2 : RI2.BC->getBinaryFunctions()) { 252a34c753fSRafael Auler const BinaryFunction &Function2 = BFI2.second; 253a34c753fSRafael Auler StringRef LTOName; 254a34c753fSRafael Auler bool Match = false; 255253b8f0aSAmir Ayupov for (const StringRef &Name : Function2.getNames()) { 256a34c753fSRafael Auler auto Iter = NameLookup.find(Name); 25715d1e517SAmir Ayupov if (std::optional<StringRef> OptionalLTOName = getLTOCommonName(Name)) 258a34c753fSRafael Auler LTOName = *OptionalLTOName; 259a34c753fSRafael Auler if (Iter == NameLookup.end()) 260a34c753fSRafael Auler continue; 261a34c753fSRafael Auler FuncMap.insert(std::make_pair<>(&Function2, Iter->second)); 262a34c753fSRafael Auler Bin1MappedFuncs.insert(Iter->second); 263a34c753fSRafael Auler Bin2MappedFuncs.insert(&Function2); 264a34c753fSRafael Auler if (Function2.hasValidProfile() && Iter->second->hasValidProfile()) { 265a34c753fSRafael Auler ++BothHaveProfile; 266a34c753fSRafael Auler Bin1ProfiledMapped.insert(Iter->second); 267a34c753fSRafael Auler } 268a34c753fSRafael Auler Match = true; 269a34c753fSRafael Auler break; 270a34c753fSRafael Auler } 271a34c753fSRafael Auler if (Match || !Function2.hasCFG()) 272a34c753fSRafael Auler continue; 273a34c753fSRafael Auler auto Iter = HashLookup.find(Function2.computeHash(/*UseDFS*/ true)); 274a34c753fSRafael Auler if (Iter != HashLookup.end()) { 275a34c753fSRafael Auler FuncMap.insert(std::make_pair<>(&Function2, Iter->second)); 276a34c753fSRafael Auler Bin1MappedFuncs.insert(Iter->second); 277a34c753fSRafael Auler Bin2MappedFuncs.insert(&Function2); 278a34c753fSRafael Auler if (Function2.hasValidProfile() && Iter->second->hasValidProfile()) { 279a34c753fSRafael Auler ++BothHaveProfile; 280a34c753fSRafael Auler Bin1ProfiledMapped.insert(Iter->second); 281a34c753fSRafael Auler } 282a34c753fSRafael Auler continue; 283a34c753fSRafael Auler } 284a34c753fSRafael Auler if (LTOName.empty()) 285a34c753fSRafael Auler continue; 286a34c753fSRafael Auler auto LTOIter = LTONameLookup1.find(LTOName); 287a34c753fSRafael Auler if (LTOIter != LTONameLookup1.end()) { 288a34c753fSRafael Auler FuncMap.insert(std::make_pair<>(&Function2, LTOIter->second)); 289a34c753fSRafael Auler Bin1MappedFuncs.insert(LTOIter->second); 290a34c753fSRafael Auler Bin2MappedFuncs.insert(&Function2); 291a34c753fSRafael Auler if (Function2.hasValidProfile() && LTOIter->second->hasValidProfile()) { 292a34c753fSRafael Auler ++BothHaveProfile; 293a34c753fSRafael Auler Bin1ProfiledMapped.insert(LTOIter->second); 294a34c753fSRafael Auler } 295a34c753fSRafael Auler } 296a34c753fSRafael Auler } 2975fb59e74SAmir Ayupov PrintProgramStats PPS; 298a34c753fSRafael Auler outs() << "* BOLT-DIFF: Starting print program stats pass for binary 1\n"; 29952cf0711SAmir Ayupov RI1.BC->logBOLTErrorsAndQuitOnFatal(PPS.runOnFunctions(*RI1.BC)); 300a34c753fSRafael Auler outs() << "* BOLT-DIFF: Starting print program stats pass for binary 2\n"; 30152cf0711SAmir Ayupov RI1.BC->logBOLTErrorsAndQuitOnFatal(PPS.runOnFunctions(*RI2.BC)); 302a34c753fSRafael Auler outs() << "=====\n"; 303a34c753fSRafael Auler outs() << "Inputs share " << BothHaveProfile 304a34c753fSRafael Auler << " functions with valid profile.\n"; 305a34c753fSRafael Auler if (opts::PrintProfiledUnmapped) { 306a34c753fSRafael Auler outs() << "\nFunctions in profile 1 that are missing in the profile 2:\n"; 307a34c753fSRafael Auler std::vector<const BinaryFunction *> Unmapped; 308a34c753fSRafael Auler for (const auto &BFI : RI1.BC->getBinaryFunctions()) { 309a34c753fSRafael Auler const BinaryFunction &Function = BFI.second; 310a34c753fSRafael Auler if (!Function.hasValidProfile() || Bin1ProfiledMapped.count(&Function)) 311a34c753fSRafael Auler continue; 312a34c753fSRafael Auler Unmapped.emplace_back(&Function); 313a34c753fSRafael Auler } 314d2c87699SAmir Ayupov llvm::sort(Unmapped, 315a34c753fSRafael Auler [&](const BinaryFunction *A, const BinaryFunction *B) { 316a34c753fSRafael Auler return A->getFunctionScore() > B->getFunctionScore(); 317a34c753fSRafael Auler }); 318a34c753fSRafael Auler for (const BinaryFunction *Function : Unmapped) { 319a34c753fSRafael Auler outs() << Function->getPrintName() << " : "; 320a34c753fSRafael Auler outs() << Function->getFunctionScore() << "\n"; 321a34c753fSRafael Auler } 322a34c753fSRafael Auler outs() << "=====\n"; 323a34c753fSRafael Auler } 324a34c753fSRafael Auler } 325a34c753fSRafael Auler 326a34c753fSRafael Auler /// Check if opcodes in BB1 match those in BB2 327a34c753fSRafael Auler bool compareBBs(const BinaryBasicBlock &BB1, 328a34c753fSRafael Auler const BinaryBasicBlock &BB2) const { 329a34c753fSRafael Auler auto Iter1 = BB1.begin(); 330a34c753fSRafael Auler auto Iter2 = BB2.begin(); 331a34c753fSRafael Auler if ((Iter1 == BB1.end() && Iter2 != BB2.end()) || 332a34c753fSRafael Auler (Iter1 != BB1.end() && Iter2 == BB2.end())) 333a34c753fSRafael Auler return false; 334a34c753fSRafael Auler 335a34c753fSRafael Auler while (Iter1 != BB1.end()) { 33640c2e0faSMaksim Panchenko if (Iter2 == BB2.end() || Iter1->getOpcode() != Iter2->getOpcode()) 337a34c753fSRafael Auler return false; 338a34c753fSRafael Auler 339a34c753fSRafael Auler ++Iter1; 340a34c753fSRafael Auler ++Iter2; 341a34c753fSRafael Auler } 342a34c753fSRafael Auler 343a34c753fSRafael Auler if (Iter2 != BB2.end()) 344a34c753fSRafael Auler return false; 345a34c753fSRafael Auler return true; 346a34c753fSRafael Auler } 347a34c753fSRafael Auler 348a34c753fSRafael Auler /// For a function in binary 2 that matched one in binary 1, now match each 349a34c753fSRafael Auler /// individual basic block in it to its corresponding blocks in binary 1. 350a34c753fSRafael Auler /// Also match each edge in binary 2 to the corresponding ones in binary 1. 351a34c753fSRafael Auler void matchBasicBlocks() { 352a34c753fSRafael Auler for (const auto &MapEntry : FuncMap) { 353a34c753fSRafael Auler const BinaryFunction *const &Func1 = MapEntry.second; 354a34c753fSRafael Auler const BinaryFunction *const &Func2 = MapEntry.first; 355a34c753fSRafael Auler 3568477bc67SFabian Parzefall auto Iter1 = Func1->getLayout().block_begin(); 3578477bc67SFabian Parzefall auto Iter2 = Func2->getLayout().block_begin(); 358a34c753fSRafael Auler 359a34c753fSRafael Auler bool Match = true; 360a34c753fSRafael Auler std::map<const BinaryBasicBlock *, const BinaryBasicBlock *> Map; 361a34c753fSRafael Auler std::map<double, std::pair<EdgeTy, EdgeTy>> EMap; 3628477bc67SFabian Parzefall while (Iter1 != Func1->getLayout().block_end()) { 3638477bc67SFabian Parzefall if (Iter2 == Func2->getLayout().block_end()) { 364a34c753fSRafael Auler Match = false; 365a34c753fSRafael Auler break; 366a34c753fSRafael Auler } 367a34c753fSRafael Auler if (!compareBBs(**Iter1, **Iter2)) { 368a34c753fSRafael Auler Match = false; 369a34c753fSRafael Auler break; 370a34c753fSRafael Auler } 371a34c753fSRafael Auler Map.insert(std::make_pair<>(*Iter2, *Iter1)); 372a34c753fSRafael Auler 373a34c753fSRafael Auler auto SuccIter1 = (*Iter1)->succ_begin(); 374a34c753fSRafael Auler auto SuccIter2 = (*Iter2)->succ_begin(); 375a34c753fSRafael Auler auto BIIter1 = (*Iter1)->branch_info_begin(); 376a34c753fSRafael Auler auto BIIter2 = (*Iter2)->branch_info_begin(); 377a34c753fSRafael Auler while (SuccIter1 != (*Iter1)->succ_end()) { 378a34c753fSRafael Auler if (SuccIter2 == (*Iter2)->succ_end()) { 379a34c753fSRafael Auler Match = false; 380a34c753fSRafael Auler break; 381a34c753fSRafael Auler } 382a34c753fSRafael Auler const double ScoreEdge1 = getNormalizedScore(BIIter1, RI1); 383a34c753fSRafael Auler const double ScoreEdge2 = getNormalizedScore(BIIter2, RI2); 384a34c753fSRafael Auler EMap.insert(std::make_pair<>( 385a34c753fSRafael Auler std::abs(ScoreEdge2 - ScoreEdge1), 386a34c753fSRafael Auler std::make_pair<>( 387a34c753fSRafael Auler std::make_tuple<>(*Iter2, *SuccIter2, ScoreEdge2), 388a34c753fSRafael Auler std::make_tuple<>(*Iter1, *SuccIter1, ScoreEdge1)))); 389a34c753fSRafael Auler 390a34c753fSRafael Auler ++SuccIter1; 391a34c753fSRafael Auler ++SuccIter2; 392a34c753fSRafael Auler ++BIIter1; 393a34c753fSRafael Auler ++BIIter2; 394a34c753fSRafael Auler } 395a34c753fSRafael Auler if (SuccIter2 != (*Iter2)->succ_end()) 396a34c753fSRafael Auler Match = false; 397a34c753fSRafael Auler if (!Match) 398a34c753fSRafael Auler break; 399a34c753fSRafael Auler 400a34c753fSRafael Auler BBToFuncMap[*Iter1] = Func1; 401a34c753fSRafael Auler BBToFuncMap[*Iter2] = Func2; 402a34c753fSRafael Auler ++Iter1; 403a34c753fSRafael Auler ++Iter2; 404a34c753fSRafael Auler } 4058477bc67SFabian Parzefall if (!Match || Iter2 != Func2->getLayout().block_end()) 406a34c753fSRafael Auler continue; 407a34c753fSRafael Auler 408a34c753fSRafael Auler BBMap.insert(Map.begin(), Map.end()); 409a34c753fSRafael Auler EdgeMap.insert(EMap.begin(), EMap.end()); 410a34c753fSRafael Auler } 411a34c753fSRafael Auler } 412a34c753fSRafael Auler 413a34c753fSRafael Auler /// Print the largest differences in basic block performance from binary 1 414a34c753fSRafael Auler /// to binary 2 415a34c753fSRafael Auler void reportHottestBBDiffs() { 416a34c753fSRafael Auler std::map<double, const BinaryBasicBlock *> LargestDiffs; 417a34c753fSRafael Auler for (const auto &MapEntry : BBMap) { 418a34c753fSRafael Auler const BinaryBasicBlock *BB2 = MapEntry.first; 419a34c753fSRafael Auler const BinaryBasicBlock *BB1 = MapEntry.second; 420a34c753fSRafael Auler LargestDiffs.insert( 421a34c753fSRafael Auler std::make_pair<>(std::abs(getNormalizedScore(*BB2, RI2) - 422a34c753fSRafael Auler getNormalizedScore(*BB1, RI1)), 423a34c753fSRafael Auler BB2)); 424a34c753fSRafael Auler } 425a34c753fSRafael Auler 426a34c753fSRafael Auler unsigned Printed = 0; 427a34c753fSRafael Auler setTitleColor(); 428a34c753fSRafael Auler outs() 429a34c753fSRafael Auler << "\nTop " << opts::DisplayCount 430a34c753fSRafael Auler << " largest differences in basic block performance bin 2 -> bin 1:\n"; 431a34c753fSRafael Auler outs() << "=========================================================\n"; 432a34c753fSRafael Auler setRegularColor(); 433a34c753fSRafael Auler outs() << " * Functions with different contents do not appear here\n\n"; 434f40d25ddSAmir Ayupov for (const BinaryBasicBlock *BB2 : 435f40d25ddSAmir Ayupov llvm::make_second_range(llvm::reverse(LargestDiffs))) { 436a34c753fSRafael Auler const double Score2 = getNormalizedScore(*BB2, RI2); 437a34c753fSRafael Auler const double Score1 = getNormalizedScore(*BBMap[BB2], RI1); 438c03e6511SAmir Ayupov const BinaryFunction *Func = BBToFuncMap[BB2]; 439c03e6511SAmir Ayupov if (opts::SkipNonSimple && !Func->isSimple()) 440c03e6511SAmir Ayupov continue; 441c03e6511SAmir Ayupov outs() << "BB " << BB2->getName() << " from " << Func->getDemangledName() 442a34c753fSRafael Auler << "\n\tScore bin1 = " << format("%.4f", Score1 * 100.0) 443a34c753fSRafael Auler << "%\n\tScore bin2 = " << format("%.4f", Score2 * 100.0); 444a34c753fSRafael Auler outs() << "%\t(Difference: "; 445a34c753fSRafael Auler printColoredPercentage((Score2 - Score1) * 100.0); 446a34c753fSRafael Auler outs() << ")\n"; 447a34c753fSRafael Auler if (opts::PrintDiffBBs) { 448a34c753fSRafael Auler setLightColor(); 449a34c753fSRafael Auler BB2->dump(); 450a34c753fSRafael Auler setRegularColor(); 451a34c753fSRafael Auler } 452a34c753fSRafael Auler if (Printed++ == opts::DisplayCount) 453a34c753fSRafael Auler break; 454a34c753fSRafael Auler } 455a34c753fSRafael Auler } 456a34c753fSRafael Auler 457a34c753fSRafael Auler /// Print the largest differences in edge counts from one binary to another 458a34c753fSRafael Auler void reportHottestEdgeDiffs() { 459a34c753fSRafael Auler unsigned Printed = 0; 460a34c753fSRafael Auler setTitleColor(); 46140c2e0faSMaksim Panchenko outs() << "\nTop " << opts::DisplayCount 462a34c753fSRafael Auler << " largest differences in edge hotness bin 2 -> bin 1:\n"; 463a34c753fSRafael Auler outs() << "=========================================================\n"; 464a34c753fSRafael Auler setRegularColor(); 465a34c753fSRafael Auler outs() << " * Functions with different contents do not appear here\n"; 466f40d25ddSAmir Ayupov for (std::pair<EdgeTy, EdgeTy> &EI : 467f40d25ddSAmir Ayupov llvm::make_second_range(llvm::reverse(EdgeMap))) { 468f40d25ddSAmir Ayupov EdgeTy &Edge2 = EI.first; 469f40d25ddSAmir Ayupov EdgeTy &Edge1 = EI.second; 470a34c753fSRafael Auler const double Score2 = std::get<2>(Edge2); 471a34c753fSRafael Auler const double Score1 = std::get<2>(Edge1); 472c03e6511SAmir Ayupov const BinaryFunction *Func = BBToFuncMap[std::get<0>(Edge2)]; 473c03e6511SAmir Ayupov if (opts::SkipNonSimple && !Func->isSimple()) 474c03e6511SAmir Ayupov continue; 475a34c753fSRafael Auler outs() << "Edge (" << std::get<0>(Edge2)->getName() << " -> " 476a34c753fSRafael Auler << std::get<1>(Edge2)->getName() << ") in " 477c03e6511SAmir Ayupov << Func->getDemangledName() 478a34c753fSRafael Auler << "\n\tScore bin1 = " << format("%.4f", Score1 * 100.0) 479a34c753fSRafael Auler << "%\n\tScore bin2 = " << format("%.4f", Score2 * 100.0); 480a34c753fSRafael Auler outs() << "%\t(Difference: "; 481a34c753fSRafael Auler printColoredPercentage((Score2 - Score1) * 100.0); 482a34c753fSRafael Auler outs() << ")\n"; 483a34c753fSRafael Auler if (opts::PrintDiffBBs) { 484a34c753fSRafael Auler setLightColor(); 485a34c753fSRafael Auler std::get<0>(Edge2)->dump(); 486a34c753fSRafael Auler std::get<1>(Edge2)->dump(); 487a34c753fSRafael Auler setRegularColor(); 488a34c753fSRafael Auler } 489a34c753fSRafael Auler if (Printed++ == opts::DisplayCount) 490a34c753fSRafael Auler break; 491a34c753fSRafael Auler } 492a34c753fSRafael Auler } 493a34c753fSRafael Auler 494a34c753fSRafael Auler /// For LTO functions sharing the same prefix (for example, func1.lto_priv.1 495a34c753fSRafael Auler /// and func1.lto_priv.2 share the func1.lto_priv prefix), compute aggregated 496a34c753fSRafael Auler /// scores for them. This is used to avoid reporting all LTO functions as 497a34c753fSRafael Auler /// having a large difference in performance because hotness shifted from 498a34c753fSRafael Auler /// LTO variant 1 to variant 2, even though they represent the same function. 499a34c753fSRafael Auler void computeAggregatedLTOScore() { 500a34c753fSRafael Auler for (const auto &BFI : RI1.BC->getBinaryFunctions()) { 501a34c753fSRafael Auler const BinaryFunction &Function = BFI.second; 502a34c753fSRafael Auler double Score = getNormalizedScore(Function, RI1); 503a34c753fSRafael Auler auto Iter = LTOMap1.find(&Function); 504a34c753fSRafael Auler if (Iter == LTOMap1.end()) 505a34c753fSRafael Auler continue; 506a34c753fSRafael Auler LTOAggregatedScore1[Iter->second] += Score; 507a34c753fSRafael Auler } 508a34c753fSRafael Auler 509a34c753fSRafael Auler double UnmappedScore = 0; 510a34c753fSRafael Auler for (const auto &BFI : RI2.BC->getBinaryFunctions()) { 511a34c753fSRafael Auler const BinaryFunction &Function = BFI.second; 512a34c753fSRafael Auler bool Matched = FuncMap.find(&Function) != FuncMap.end(); 513a34c753fSRafael Auler double Score = getNormalizedScore(Function, RI2); 514a34c753fSRafael Auler auto Iter = LTOMap2.find(&Function); 515a34c753fSRafael Auler if (Iter == LTOMap2.end()) { 516a34c753fSRafael Auler if (!Matched) 517a34c753fSRafael Auler UnmappedScore += Score; 518a34c753fSRafael Auler continue; 519a34c753fSRafael Auler } 520a34c753fSRafael Auler LTOAggregatedScore2[Iter->second] += Score; 521a34c753fSRafael Auler if (FuncMap.find(Iter->second) == FuncMap.end()) 522a34c753fSRafael Auler UnmappedScore += Score; 523a34c753fSRafael Auler } 524a34c753fSRafael Auler int64_t Unmapped = 525a34c753fSRafael Auler RI2.BC->getBinaryFunctions().size() - Bin2MappedFuncs.size(); 526a34c753fSRafael Auler outs() << "BOLT-DIFF: " << Unmapped 527a34c753fSRafael Auler << " functions in Binary2 have no correspondence to any other " 528a34c753fSRafael Auler "function in Binary1.\n"; 529a34c753fSRafael Auler 530a34c753fSRafael Auler // Print the hotness score of functions in binary 2 that were not matched 531a34c753fSRafael Auler // to any function in binary 1 532a34c753fSRafael Auler outs() << "BOLT-DIFF: These unmapped functions in Binary2 represent " 533a34c753fSRafael Auler << format("%.2f", UnmappedScore * 100.0) << "% of execution.\n"; 534a34c753fSRafael Auler } 535a34c753fSRafael Auler 536a34c753fSRafael Auler /// Print the largest hotness differences from binary 2 to binary 1 537a34c753fSRafael Auler void reportHottestFuncDiffs() { 538a34c753fSRafael Auler std::multimap<double, decltype(FuncMap)::value_type> LargestDiffs; 539a34c753fSRafael Auler for (const auto &MapEntry : FuncMap) { 540a34c753fSRafael Auler const BinaryFunction *const &Func1 = MapEntry.second; 541a34c753fSRafael Auler const BinaryFunction *const &Func2 = MapEntry.first; 542a34c753fSRafael Auler double Score1 = getNormalizedScore(*Func1, RI1); 543a34c753fSRafael Auler auto Iter1 = LTOMap1.find(Func1); 544ee0e9ccbSMaksim Panchenko if (Iter1 != LTOMap1.end()) 545a34c753fSRafael Auler Score1 = LTOAggregatedScore1[Iter1->second]; 546a34c753fSRafael Auler double Score2 = getNormalizedScore(*Func2, RI2); 547a34c753fSRafael Auler auto Iter2 = LTOMap2.find(Func2); 548ee0e9ccbSMaksim Panchenko if (Iter2 != LTOMap2.end()) 549a34c753fSRafael Auler Score2 = LTOAggregatedScore2[Iter2->second]; 550a34c753fSRafael Auler if (Score1 == 0.0 || Score2 == 0.0) 551a34c753fSRafael Auler continue; 552c03e6511SAmir Ayupov if (opts::SkipNonSimple && !Func1->isSimple() && !Func2->isSimple()) 553c03e6511SAmir Ayupov continue; 554a34c753fSRafael Auler LargestDiffs.insert( 555a34c753fSRafael Auler std::make_pair<>(std::abs(Score1 - Score2), MapEntry)); 556a34c753fSRafael Auler ScoreMap[Func2] = std::make_pair<>(Score1, Score2); 557a34c753fSRafael Auler } 558a34c753fSRafael Auler 559a34c753fSRafael Auler unsigned Printed = 0; 560a34c753fSRafael Auler setTitleColor(); 561a34c753fSRafael Auler outs() << "\nTop " << opts::DisplayCount 562a34c753fSRafael Auler << " largest differences in performance bin 2 -> bin 1:\n"; 563a34c753fSRafael Auler outs() << "=========================================================\n"; 564a34c753fSRafael Auler setRegularColor(); 565f40d25ddSAmir Ayupov for (decltype(this->FuncMap)::value_type &MapEntry : 566f40d25ddSAmir Ayupov llvm::make_second_range(llvm::reverse(LargestDiffs))) { 567a34c753fSRafael Auler if (opts::IgnoreUnchanged && 568a34c753fSRafael Auler MapEntry.second->computeHash(/*UseDFS=*/true) == 569a34c753fSRafael Auler MapEntry.first->computeHash(/*UseDFS=*/true)) 570a34c753fSRafael Auler continue; 571a34c753fSRafael Auler const std::pair<double, double> &Scores = ScoreMap[MapEntry.first]; 572a34c753fSRafael Auler outs() << "Function " << MapEntry.first->getDemangledName(); 573a34c753fSRafael Auler if (MapEntry.first->getDemangledName() != 574a34c753fSRafael Auler MapEntry.second->getDemangledName()) 575a34c753fSRafael Auler outs() << "\nmatched " << MapEntry.second->getDemangledName(); 576a34c753fSRafael Auler outs() << "\n\tScore bin1 = " << format("%.2f", Scores.first * 100.0) 577a34c753fSRafael Auler << "%\n\tScore bin2 = " << format("%.2f", Scores.second * 100.0) 578a34c753fSRafael Auler << "%\t(Difference: "; 579a34c753fSRafael Auler printColoredPercentage((Scores.second - Scores.first) * 100.0); 580a34c753fSRafael Auler outs() << ")"; 581a34c753fSRafael Auler if (MapEntry.second->computeHash(/*UseDFS=*/true) != 582a34c753fSRafael Auler MapEntry.first->computeHash(/*UseDFS=*/true)) { 583a34c753fSRafael Auler outs() << "\t[Functions have different contents]"; 584a34c753fSRafael Auler if (opts::PrintDiffCFG) { 585a34c753fSRafael Auler outs() << "\n *** CFG for function in binary 1:\n"; 586a34c753fSRafael Auler setLightColor(); 587a34c753fSRafael Auler MapEntry.second->dump(); 588a34c753fSRafael Auler setRegularColor(); 589a34c753fSRafael Auler outs() << "\n *** CFG for function in binary 2:\n"; 590a34c753fSRafael Auler setLightColor(); 591a34c753fSRafael Auler MapEntry.first->dump(); 592a34c753fSRafael Auler setRegularColor(); 593a34c753fSRafael Auler } 594a34c753fSRafael Auler } 595a34c753fSRafael Auler outs() << "\n"; 596a34c753fSRafael Auler if (Printed++ == opts::DisplayCount) 597a34c753fSRafael Auler break; 598a34c753fSRafael Auler } 599a34c753fSRafael Auler } 600a34c753fSRafael Auler 601a34c753fSRafael Auler /// Print hottest functions from each binary 602a34c753fSRafael Auler void reportHottestFuncs() { 603a34c753fSRafael Auler unsigned Printed = 0; 604a34c753fSRafael Auler setTitleColor(); 605a34c753fSRafael Auler outs() << "\nTop " << opts::DisplayCount 606a34c753fSRafael Auler << " hottest functions in binary 2:\n"; 607a34c753fSRafael Auler outs() << "=====================================\n"; 608a34c753fSRafael Auler setRegularColor(); 609f40d25ddSAmir Ayupov for (std::pair<const double, const BinaryFunction *> &MapEntry : 610f40d25ddSAmir Ayupov llvm::reverse(LargestBin2)) { 611a34c753fSRafael Auler outs() << "Function " << MapEntry.second->getDemangledName() << "\n"; 612a34c753fSRafael Auler auto Iter = ScoreMap.find(MapEntry.second); 613ee0e9ccbSMaksim Panchenko if (Iter != ScoreMap.end()) 614a34c753fSRafael Auler outs() << "\tScore bin1 = " 615a34c753fSRafael Auler << format("%.2f", Iter->second.first * 100.0) << "%\n"; 616a34c753fSRafael Auler outs() << "\tScore bin2 = " << format("%.2f", MapEntry.first * 100.0) 617a34c753fSRafael Auler << "%\n"; 618a34c753fSRafael Auler if (Printed++ == opts::DisplayCount) 619a34c753fSRafael Auler break; 620a34c753fSRafael Auler } 621a34c753fSRafael Auler 622a34c753fSRafael Auler Printed = 0; 623a34c753fSRafael Auler setTitleColor(); 624a34c753fSRafael Auler outs() << "\nTop " << opts::DisplayCount 625a34c753fSRafael Auler << " hottest functions in binary 1:\n"; 626a34c753fSRafael Auler outs() << "=====================================\n"; 627a34c753fSRafael Auler setRegularColor(); 628f40d25ddSAmir Ayupov for (const std::pair<const double, const BinaryFunction *> &MapEntry : 629f40d25ddSAmir Ayupov llvm::reverse(LargestBin1)) { 630a34c753fSRafael Auler outs() << "Function " << MapEntry.second->getDemangledName() 631a34c753fSRafael Auler << "\n\tScore bin1 = " << format("%.2f", MapEntry.first * 100.0) 632a34c753fSRafael Auler << "%\n"; 633a34c753fSRafael Auler if (Printed++ == opts::DisplayCount) 634a34c753fSRafael Auler break; 635a34c753fSRafael Auler } 636a34c753fSRafael Auler } 637a34c753fSRafael Auler 638a34c753fSRafael Auler /// Print functions in binary 2 that did not match anything in binary 1. 639a34c753fSRafael Auler /// Unfortunately, in an LTO build, even a small change can lead to several 640a34c753fSRafael Auler /// LTO variants being unmapped, corresponding to local functions that never 641a34c753fSRafael Auler /// appear in one of the binaries because they were previously inlined. 642a34c753fSRafael Auler void reportUnmapped() { 643a34c753fSRafael Auler outs() << "List of functions from binary 2 that were not matched with any " 644a34c753fSRafael Auler << "function in binary 1:\n"; 645a34c753fSRafael Auler for (const auto &BFI2 : RI2.BC->getBinaryFunctions()) { 646a34c753fSRafael Auler const BinaryFunction &Function2 = BFI2.second; 647a34c753fSRafael Auler if (Bin2MappedFuncs.count(&Function2)) 648a34c753fSRafael Auler continue; 649a34c753fSRafael Auler outs() << Function2.getPrintName() << "\n"; 650a34c753fSRafael Auler } 651a34c753fSRafael Auler } 652a34c753fSRafael Auler 653a34c753fSRafael Auler public: 654a34c753fSRafael Auler /// Main entry point: coordinate all tasks necessary to compare two binaries 655a34c753fSRafael Auler void compareAndReport() { 656a34c753fSRafael Auler buildLookupMaps(); 657a34c753fSRafael Auler matchFunctions(); 658a34c753fSRafael Auler if (opts::IgnoreLTOSuffix) 659a34c753fSRafael Auler computeAggregatedLTOScore(); 660a34c753fSRafael Auler matchBasicBlocks(); 661a34c753fSRafael Auler reportHottestFuncDiffs(); 662a34c753fSRafael Auler reportHottestBBDiffs(); 663a34c753fSRafael Auler reportHottestEdgeDiffs(); 664a34c753fSRafael Auler reportHottestFuncs(); 665a34c753fSRafael Auler if (!opts::PrintUnmapped) 666a34c753fSRafael Auler return; 667a34c753fSRafael Auler reportUnmapped(); 668a34c753fSRafael Auler } 669a34c753fSRafael Auler 670a34c753fSRafael Auler RewriteInstanceDiff(RewriteInstance &RI1, RewriteInstance &RI2) 671a34c753fSRafael Auler : RI1(RI1), RI2(RI2) { 672a34c753fSRafael Auler compareAndReport(); 673a34c753fSRafael Auler } 674a34c753fSRafael Auler 675a34c753fSRafael Auler }; 676a34c753fSRafael Auler 6771a2f8336Sspaette } // end namespace bolt 678a34c753fSRafael Auler } // end namespace llvm 679a34c753fSRafael Auler 680a34c753fSRafael Auler void RewriteInstance::compare(RewriteInstance &RI2) { 681a34c753fSRafael Auler outs() << "BOLT-DIFF: ======== Binary1 vs. Binary2 ========\n"; 682a34c753fSRafael Auler outs() << "Trace for binary 1 has " << this->getTotalScore() 683a34c753fSRafael Auler << " instructions executed.\n"; 684a34c753fSRafael Auler outs() << "Trace for binary 2 has " << RI2.getTotalScore() 685a34c753fSRafael Auler << " instructions executed.\n"; 686a34c753fSRafael Auler if (opts::NormalizeByBin1) { 68740c2e0faSMaksim Panchenko double Diff2to1 = 68840c2e0faSMaksim Panchenko static_cast<double>(RI2.getTotalScore() - this->getTotalScore()) / 689a34c753fSRafael Auler this->getTotalScore(); 690a34c753fSRafael Auler outs() << "Binary2 change in score with respect to Binary1: "; 691a34c753fSRafael Auler printColoredPercentage(Diff2to1 * 100.0); 692a34c753fSRafael Auler outs() << "\n"; 693a34c753fSRafael Auler } 694a34c753fSRafael Auler 695a34c753fSRafael Auler if (!this->getTotalScore() || !RI2.getTotalScore()) { 696a34c753fSRafael Auler outs() << "BOLT-DIFF: Both binaries must have recorded activity in known " 697a34c753fSRafael Auler "functions.\n"; 698a34c753fSRafael Auler return; 699a34c753fSRafael Auler } 700a34c753fSRafael Auler 701a34c753fSRafael Auler // Pre-pass ICF 702*3c357a49SAlexander Yermolovich if (opts::ICF != IdenticalCodeFolding::ICFLevel::None) { 703a34c753fSRafael Auler IdenticalCodeFolding ICF(opts::NeverPrint); 704a34c753fSRafael Auler outs() << "BOLT-DIFF: Starting ICF pass for binary 1"; 70552cf0711SAmir Ayupov BC->logBOLTErrorsAndQuitOnFatal(ICF.runOnFunctions(*BC)); 706a34c753fSRafael Auler outs() << "BOLT-DIFF: Starting ICF pass for binary 2"; 70752cf0711SAmir Ayupov BC->logBOLTErrorsAndQuitOnFatal(ICF.runOnFunctions(*RI2.BC)); 708a34c753fSRafael Auler } 709a34c753fSRafael Auler 710a34c753fSRafael Auler RewriteInstanceDiff RID(*this, RI2); 711a34c753fSRafael Auler } 712