diff options
author | Adam Nemet <anemet@apple.com> | 2017-12-01 18:12:29 +0000 |
---|---|---|
committer | Adam Nemet <anemet@apple.com> | 2017-12-01 18:12:29 +0000 |
commit | 57783730fd3be5f35bff3444f90de9c44e150bb3 (patch) | |
tree | 0cd745bd25ab324bc97f97fe74414410dffadbd9 /llvm/lib/Analysis/OptimizationRemarkEmitter.cpp | |
parent | 94dca7c7eacd363041d747fa1dd2393a5109c67f (diff) | |
download | bcm5719-llvm-57783730fd3be5f35bff3444f90de9c44e150bb3.tar.gz bcm5719-llvm-57783730fd3be5f35bff3444f90de9c44e150bb3.zip |
Revert "[opt-remarks] If hotness threshold is set, ignore remarks without hotness"
This reverts commit r319556.
Something is not working with this when used with sample-based profiling.
Investigating...
llvm-svn: 319562
Diffstat (limited to 'llvm/lib/Analysis/OptimizationRemarkEmitter.cpp')
-rw-r--r-- | llvm/lib/Analysis/OptimizationRemarkEmitter.cpp | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/llvm/lib/Analysis/OptimizationRemarkEmitter.cpp b/llvm/lib/Analysis/OptimizationRemarkEmitter.cpp index 4ff7611086e..cd6a9366801 100644 --- a/llvm/lib/Analysis/OptimizationRemarkEmitter.cpp +++ b/llvm/lib/Analysis/OptimizationRemarkEmitter.cpp @@ -75,10 +75,11 @@ void OptimizationRemarkEmitter::emit( DiagnosticInfoOptimizationBase &OptDiagBase) { auto &OptDiag = cast<DiagnosticInfoIROptimization>(OptDiagBase); computeHotness(OptDiag); - - // Only emit it if its hotness meets the threshold. - if (OptDiag.getHotness().getValueOr(0) < - F->getContext().getDiagnosticsHotnessThreshold()) { + // If a diagnostic has a hotness value, then only emit it if its hotness + // meets the threshold. + if (OptDiag.getHotness() && + *OptDiag.getHotness() < + F->getContext().getDiagnosticsHotnessThreshold()) { return; } |