Adam Nemet | aad8160 | 2016-07-15 17:23:20 +0000 | [diff] [blame] | 1 | //===- OptimizationDiagnosticInfo.cpp - Optimization Diagnostic -*- C++ -*-===// |
| 2 | // |
| 3 | // The LLVM Compiler Infrastructure |
| 4 | // |
| 5 | // This file is distributed under the University of Illinois Open Source |
| 6 | // License. See LICENSE.TXT for details. |
| 7 | // |
| 8 | //===----------------------------------------------------------------------===// |
| 9 | // |
| 10 | // Optimization diagnostic interfaces. It's packaged as an analysis pass so |
| 11 | // that by using this service passes become dependent on BFI as well. BFI is |
| 12 | // used to compute the "hotness" of the diagnostic message. |
| 13 | //===----------------------------------------------------------------------===// |
| 14 | |
| 15 | #include "llvm/Analysis/OptimizationDiagnosticInfo.h" |
Adam Nemet | 896c09b | 2016-08-10 00:44:44 +0000 | [diff] [blame] | 16 | #include "llvm/Analysis/BranchProbabilityInfo.h" |
Adam Nemet | aad8160 | 2016-07-15 17:23:20 +0000 | [diff] [blame] | 17 | #include "llvm/Analysis/LazyBlockFrequencyInfo.h" |
| 18 | #include "llvm/Analysis/LoopInfo.h" |
Adam Nemet | a62b7e1 | 2016-09-27 20:55:07 +0000 | [diff] [blame] | 19 | #include "llvm/IR/DebugInfo.h" |
Adam Nemet | aad8160 | 2016-07-15 17:23:20 +0000 | [diff] [blame] | 20 | #include "llvm/IR/DiagnosticInfo.h" |
Adam Nemet | 896c09b | 2016-08-10 00:44:44 +0000 | [diff] [blame] | 21 | #include "llvm/IR/Dominators.h" |
Adam Nemet | aad8160 | 2016-07-15 17:23:20 +0000 | [diff] [blame] | 22 | #include "llvm/IR/LLVMContext.h" |
| 23 | |
| 24 | using namespace llvm; |
| 25 | |
Justin Bogner | 8281c81 | 2017-02-22 07:38:17 +0000 | [diff] [blame] | 26 | OptimizationRemarkEmitter::OptimizationRemarkEmitter(const Function *F) |
Adam Nemet | 896c09b | 2016-08-10 00:44:44 +0000 | [diff] [blame] | 27 | : F(F), BFI(nullptr) { |
Brian Gesiak | 44e5f6c | 2017-06-30 18:13:59 +0000 | [diff] [blame] | 28 | if (!F->getContext().getDiagnosticsHotnessRequested()) |
Adam Nemet | 896c09b | 2016-08-10 00:44:44 +0000 | [diff] [blame] | 29 | return; |
| 30 | |
| 31 | // First create a dominator tree. |
| 32 | DominatorTree DT; |
Justin Bogner | 8281c81 | 2017-02-22 07:38:17 +0000 | [diff] [blame] | 33 | DT.recalculate(*const_cast<Function *>(F)); |
Adam Nemet | 896c09b | 2016-08-10 00:44:44 +0000 | [diff] [blame] | 34 | |
| 35 | // Generate LoopInfo from it. |
| 36 | LoopInfo LI; |
| 37 | LI.analyze(DT); |
| 38 | |
| 39 | // Then compute BranchProbabilityInfo. |
| 40 | BranchProbabilityInfo BPI; |
| 41 | BPI.calculate(*F, LI); |
| 42 | |
| 43 | // Finally compute BFI. |
| 44 | OwnedBFI = llvm::make_unique<BlockFrequencyInfo>(*F, BPI, LI); |
| 45 | BFI = OwnedBFI.get(); |
| 46 | } |
| 47 | |
Chandler Carruth | 1ae34c3 | 2017-01-15 08:20:50 +0000 | [diff] [blame] | 48 | bool OptimizationRemarkEmitter::invalidate( |
| 49 | Function &F, const PreservedAnalyses &PA, |
| 50 | FunctionAnalysisManager::Invalidator &Inv) { |
| 51 | // This analysis has no state and so can be trivially preserved but it needs |
| 52 | // a fresh view of BFI if it was constructed with one. |
| 53 | if (BFI && Inv.invalidate<BlockFrequencyAnalysis>(F, PA)) |
| 54 | return true; |
| 55 | |
| 56 | // Otherwise this analysis result remains valid. |
| 57 | return false; |
| 58 | } |
| 59 | |
Adam Nemet | 6100d16 | 2016-07-20 21:44:22 +0000 | [diff] [blame] | 60 | Optional<uint64_t> OptimizationRemarkEmitter::computeHotness(const Value *V) { |
Adam Nemet | aad8160 | 2016-07-15 17:23:20 +0000 | [diff] [blame] | 61 | if (!BFI) |
| 62 | return None; |
| 63 | |
| 64 | return BFI->getBlockProfileCount(cast<BasicBlock>(V)); |
| 65 | } |
| 66 | |
Adam Nemet | a62b7e1 | 2016-09-27 20:55:07 +0000 | [diff] [blame] | 67 | namespace llvm { |
| 68 | namespace yaml { |
| 69 | |
Adam Nemet | a964066 | 2017-01-25 23:20:33 +0000 | [diff] [blame] | 70 | void MappingTraits<DiagnosticInfoOptimizationBase *>::mapping( |
| 71 | IO &io, DiagnosticInfoOptimizationBase *&OptDiag) { |
| 72 | assert(io.outputting() && "input not yet implemented"); |
Adam Nemet | a62b7e1 | 2016-09-27 20:55:07 +0000 | [diff] [blame] | 73 | |
Adam Nemet | 916923e | 2017-01-26 00:39:51 +0000 | [diff] [blame] | 74 | if (io.mapTag("!Passed", |
| 75 | (OptDiag->getKind() == DK_OptimizationRemark || |
| 76 | OptDiag->getKind() == DK_MachineOptimizationRemark))) |
Adam Nemet | a964066 | 2017-01-25 23:20:33 +0000 | [diff] [blame] | 77 | ; |
Adam Nemet | 916923e | 2017-01-26 00:39:51 +0000 | [diff] [blame] | 78 | else if (io.mapTag( |
| 79 | "!Missed", |
| 80 | (OptDiag->getKind() == DK_OptimizationRemarkMissed || |
| 81 | OptDiag->getKind() == DK_MachineOptimizationRemarkMissed))) |
Adam Nemet | a964066 | 2017-01-25 23:20:33 +0000 | [diff] [blame] | 82 | ; |
Adam Nemet | 916923e | 2017-01-26 00:39:51 +0000 | [diff] [blame] | 83 | else if (io.mapTag( |
| 84 | "!Analysis", |
| 85 | (OptDiag->getKind() == DK_OptimizationRemarkAnalysis || |
| 86 | OptDiag->getKind() == DK_MachineOptimizationRemarkAnalysis))) |
Adam Nemet | a964066 | 2017-01-25 23:20:33 +0000 | [diff] [blame] | 87 | ; |
| 88 | else if (io.mapTag("!AnalysisFPCommute", |
| 89 | OptDiag->getKind() == |
| 90 | DK_OptimizationRemarkAnalysisFPCommute)) |
| 91 | ; |
| 92 | else if (io.mapTag("!AnalysisAliasing", |
| 93 | OptDiag->getKind() == |
| 94 | DK_OptimizationRemarkAnalysisAliasing)) |
| 95 | ; |
Adam Nemet | 0bf1b86 | 2017-02-02 05:41:51 +0000 | [diff] [blame] | 96 | else if (io.mapTag("!Failure", OptDiag->getKind() == DK_OptimizationFailure)) |
| 97 | ; |
Adam Nemet | a964066 | 2017-01-25 23:20:33 +0000 | [diff] [blame] | 98 | else |
Adam Nemet | 0bf1b86 | 2017-02-02 05:41:51 +0000 | [diff] [blame] | 99 | llvm_unreachable("Unknown remark type"); |
Adam Nemet | a62b7e1 | 2016-09-27 20:55:07 +0000 | [diff] [blame] | 100 | |
Adam Nemet | a964066 | 2017-01-25 23:20:33 +0000 | [diff] [blame] | 101 | // These are read-only for now. |
Justin Bogner | d890f95 | 2017-02-18 00:42:23 +0000 | [diff] [blame] | 102 | DiagnosticLocation DL = OptDiag->getLocation(); |
Adam Nemet | a964066 | 2017-01-25 23:20:33 +0000 | [diff] [blame] | 103 | StringRef FN = |
Peter Collingbourne | 6f0ecca | 2017-05-16 00:39:01 +0000 | [diff] [blame] | 104 | GlobalValue::dropLLVMManglingEscape(OptDiag->getFunction().getName()); |
Adam Nemet | a62b7e1 | 2016-09-27 20:55:07 +0000 | [diff] [blame] | 105 | |
Adam Nemet | a964066 | 2017-01-25 23:20:33 +0000 | [diff] [blame] | 106 | StringRef PassName(OptDiag->PassName); |
| 107 | io.mapRequired("Pass", PassName); |
| 108 | io.mapRequired("Name", OptDiag->RemarkName); |
Justin Bogner | d890f95 | 2017-02-18 00:42:23 +0000 | [diff] [blame] | 109 | if (!io.outputting() || DL.isValid()) |
Adam Nemet | a964066 | 2017-01-25 23:20:33 +0000 | [diff] [blame] | 110 | io.mapOptional("DebugLoc", DL); |
| 111 | io.mapRequired("Function", FN); |
| 112 | io.mapOptional("Hotness", OptDiag->Hotness); |
| 113 | io.mapOptional("Args", OptDiag->Args); |
| 114 | } |
Adam Nemet | a62b7e1 | 2016-09-27 20:55:07 +0000 | [diff] [blame] | 115 | |
Justin Bogner | d890f95 | 2017-02-18 00:42:23 +0000 | [diff] [blame] | 116 | template <> struct MappingTraits<DiagnosticLocation> { |
| 117 | static void mapping(IO &io, DiagnosticLocation &DL) { |
Adam Nemet | a62b7e1 | 2016-09-27 20:55:07 +0000 | [diff] [blame] | 118 | assert(io.outputting() && "input not yet implemented"); |
| 119 | |
Justin Bogner | d890f95 | 2017-02-18 00:42:23 +0000 | [diff] [blame] | 120 | StringRef File = DL.getFilename(); |
Adam Nemet | a62b7e1 | 2016-09-27 20:55:07 +0000 | [diff] [blame] | 121 | unsigned Line = DL.getLine(); |
Justin Bogner | d890f95 | 2017-02-18 00:42:23 +0000 | [diff] [blame] | 122 | unsigned Col = DL.getColumn(); |
Adam Nemet | a62b7e1 | 2016-09-27 20:55:07 +0000 | [diff] [blame] | 123 | |
| 124 | io.mapRequired("File", File); |
| 125 | io.mapRequired("Line", Line); |
| 126 | io.mapRequired("Column", Col); |
| 127 | } |
| 128 | |
| 129 | static const bool flow = true; |
| 130 | }; |
| 131 | |
Adam Nemet | 0428e93 | 2016-10-04 17:05:04 +0000 | [diff] [blame] | 132 | // Implement this as a mapping for now to get proper quotation for the value. |
| 133 | template <> struct MappingTraits<DiagnosticInfoOptimizationBase::Argument> { |
| 134 | static void mapping(IO &io, DiagnosticInfoOptimizationBase::Argument &A) { |
| 135 | assert(io.outputting() && "input not yet implemented"); |
| 136 | io.mapRequired(A.Key.data(), A.Val); |
Justin Bogner | d890f95 | 2017-02-18 00:42:23 +0000 | [diff] [blame] | 137 | if (A.Loc.isValid()) |
| 138 | io.mapOptional("DebugLoc", A.Loc); |
Adam Nemet | a62b7e1 | 2016-09-27 20:55:07 +0000 | [diff] [blame] | 139 | } |
Adam Nemet | a62b7e1 | 2016-09-27 20:55:07 +0000 | [diff] [blame] | 140 | }; |
| 141 | |
| 142 | } // end namespace yaml |
| 143 | } // end namespace llvm |
| 144 | |
| 145 | LLVM_YAML_IS_SEQUENCE_VECTOR(DiagnosticInfoOptimizationBase::Argument) |
| 146 | |
| 147 | void OptimizationRemarkEmitter::computeHotness( |
Adam Nemet | 484f93d | 2017-01-25 23:20:25 +0000 | [diff] [blame] | 148 | DiagnosticInfoIROptimization &OptDiag) { |
Justin Bogner | 8281c81 | 2017-02-22 07:38:17 +0000 | [diff] [blame] | 149 | const Value *V = OptDiag.getCodeRegion(); |
Adam Nemet | a62b7e1 | 2016-09-27 20:55:07 +0000 | [diff] [blame] | 150 | if (V) |
| 151 | OptDiag.setHotness(computeHotness(V)); |
| 152 | } |
| 153 | |
Adam Nemet | 484f93d | 2017-01-25 23:20:25 +0000 | [diff] [blame] | 154 | void OptimizationRemarkEmitter::emit( |
| 155 | DiagnosticInfoOptimizationBase &OptDiagBase) { |
| 156 | auto &OptDiag = cast<DiagnosticInfoIROptimization>(OptDiagBase); |
Adam Nemet | a62b7e1 | 2016-09-27 20:55:07 +0000 | [diff] [blame] | 157 | computeHotness(OptDiag); |
Brian Gesiak | 4ef3daa | 2017-06-30 23:14:53 +0000 | [diff] [blame] | 158 | // If a diagnostic has a hotness value, then only emit it if its hotness |
| 159 | // meets the threshold. |
| 160 | if (OptDiag.getHotness() && |
| 161 | *OptDiag.getHotness() < |
| 162 | F->getContext().getDiagnosticsHotnessThreshold()) { |
| 163 | return; |
| 164 | } |
Adam Nemet | a62b7e1 | 2016-09-27 20:55:07 +0000 | [diff] [blame] | 165 | |
| 166 | yaml::Output *Out = F->getContext().getDiagnosticsOutputFile(); |
Hal Finkel | bdd6735 | 2016-10-04 18:13:45 +0000 | [diff] [blame] | 167 | if (Out) { |
Adam Nemet | 4846e66 | 2017-08-28 23:00:13 +0000 | [diff] [blame] | 168 | // For remarks the << operator takes a reference to a pointer. |
| 169 | auto *P = &OptDiagBase; |
Adam Nemet | a62b7e1 | 2016-09-27 20:55:07 +0000 | [diff] [blame] | 170 | *Out << P; |
| 171 | } |
Adam Nemet | f31b1f3 | 2017-10-04 04:26:23 +0000 | [diff] [blame^] | 172 | F->getContext().diagnose(OptDiag); |
Adam Nemet | a62b7e1 | 2016-09-27 20:55:07 +0000 | [diff] [blame] | 173 | } |
| 174 | |
Adam Nemet | 79ac42a | 2016-07-18 16:29:21 +0000 | [diff] [blame] | 175 | OptimizationRemarkEmitterWrapperPass::OptimizationRemarkEmitterWrapperPass() |
| 176 | : FunctionPass(ID) { |
| 177 | initializeOptimizationRemarkEmitterWrapperPassPass( |
| 178 | *PassRegistry::getPassRegistry()); |
| 179 | } |
| 180 | |
| 181 | bool OptimizationRemarkEmitterWrapperPass::runOnFunction(Function &Fn) { |
| 182 | BlockFrequencyInfo *BFI; |
Adam Nemet | aad8160 | 2016-07-15 17:23:20 +0000 | [diff] [blame] | 183 | |
Brian Gesiak | 44e5f6c | 2017-06-30 18:13:59 +0000 | [diff] [blame] | 184 | if (Fn.getContext().getDiagnosticsHotnessRequested()) |
Adam Nemet | aad8160 | 2016-07-15 17:23:20 +0000 | [diff] [blame] | 185 | BFI = &getAnalysis<LazyBlockFrequencyInfoPass>().getBFI(); |
| 186 | else |
| 187 | BFI = nullptr; |
| 188 | |
Adam Nemet | 79ac42a | 2016-07-18 16:29:21 +0000 | [diff] [blame] | 189 | ORE = llvm::make_unique<OptimizationRemarkEmitter>(&Fn, BFI); |
Adam Nemet | aad8160 | 2016-07-15 17:23:20 +0000 | [diff] [blame] | 190 | return false; |
| 191 | } |
| 192 | |
Adam Nemet | 79ac42a | 2016-07-18 16:29:21 +0000 | [diff] [blame] | 193 | void OptimizationRemarkEmitterWrapperPass::getAnalysisUsage( |
| 194 | AnalysisUsage &AU) const { |
Adam Nemet | aad8160 | 2016-07-15 17:23:20 +0000 | [diff] [blame] | 195 | LazyBlockFrequencyInfoPass::getLazyBFIAnalysisUsage(AU); |
| 196 | AU.setPreservesAll(); |
| 197 | } |
| 198 | |
Chandler Carruth | dab4eae | 2016-11-23 17:53:26 +0000 | [diff] [blame] | 199 | AnalysisKey OptimizationRemarkEmitterAnalysis::Key; |
Adam Nemet | 79ac42a | 2016-07-18 16:29:21 +0000 | [diff] [blame] | 200 | |
| 201 | OptimizationRemarkEmitter |
Adam Nemet | 78d1091 | 2016-07-20 21:44:18 +0000 | [diff] [blame] | 202 | OptimizationRemarkEmitterAnalysis::run(Function &F, |
Sean Silva | 36e0d01 | 2016-08-09 00:28:15 +0000 | [diff] [blame] | 203 | FunctionAnalysisManager &AM) { |
Adam Nemet | 79ac42a | 2016-07-18 16:29:21 +0000 | [diff] [blame] | 204 | BlockFrequencyInfo *BFI; |
| 205 | |
Brian Gesiak | 44e5f6c | 2017-06-30 18:13:59 +0000 | [diff] [blame] | 206 | if (F.getContext().getDiagnosticsHotnessRequested()) |
Adam Nemet | 79ac42a | 2016-07-18 16:29:21 +0000 | [diff] [blame] | 207 | BFI = &AM.getResult<BlockFrequencyAnalysis>(F); |
| 208 | else |
| 209 | BFI = nullptr; |
| 210 | |
| 211 | return OptimizationRemarkEmitter(&F, BFI); |
| 212 | } |
| 213 | |
| 214 | char OptimizationRemarkEmitterWrapperPass::ID = 0; |
Adam Nemet | aad8160 | 2016-07-15 17:23:20 +0000 | [diff] [blame] | 215 | static const char ore_name[] = "Optimization Remark Emitter"; |
| 216 | #define ORE_NAME "opt-remark-emitter" |
| 217 | |
Adam Nemet | 79ac42a | 2016-07-18 16:29:21 +0000 | [diff] [blame] | 218 | INITIALIZE_PASS_BEGIN(OptimizationRemarkEmitterWrapperPass, ORE_NAME, ore_name, |
| 219 | false, true) |
Adam Nemet | aad8160 | 2016-07-15 17:23:20 +0000 | [diff] [blame] | 220 | INITIALIZE_PASS_DEPENDENCY(LazyBFIPass) |
Adam Nemet | 79ac42a | 2016-07-18 16:29:21 +0000 | [diff] [blame] | 221 | INITIALIZE_PASS_END(OptimizationRemarkEmitterWrapperPass, ORE_NAME, ore_name, |
| 222 | false, true) |