[PATCH] D41461: Add hasProfileData() to check if a function has profile data. NFC.

Easwaran Raman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 20 12:43:39 PST 2017


eraman created this revision.
eraman added reviewers: davidxl, silvas.

This replaces calls to getEntryCount().hasValue() with hasProfileData
that does the same thing. This refactoring is useful to do before adding
synthetic function entry counts but also a useful cleanup IMO even
otherwise. I have used hasProfileData instead of hasRealProfileData as
David had earlier suggested since I think profile implies "real" and I
use the phrase "synthetic entry count" and not "synthetic profile count"
but I am fine calling it hasRealProfileData if you prefer.


https://reviews.llvm.org/D41461

Files:
  include/llvm/IR/Function.h
  lib/Analysis/ModuleSummaryAnalysis.cpp
  lib/Analysis/ProfileSummaryInfo.cpp
  lib/Bitcode/Writer/BitcodeWriter.cpp
  lib/CodeGen/MachineBlockPlacement.cpp
  lib/Transforms/IPO/PartialInlining.cpp
  lib/Transforms/Scalar/JumpThreading.cpp
  lib/Transforms/Scalar/LoopSink.cpp
  lib/Transforms/Scalar/LoopUnrollPass.cpp
  lib/Transforms/Utils/LoopUnrollPeel.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D41461.127776.patch
Type: text/x-patch
Size: 7968 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171220/b4e55c8e/attachment.bin>


More information about the llvm-commits mailing list