From b64a73cd994ce89d2b813ac522a3da3870b9c84d Mon Sep 17 00:00:00 2001 From: Rong Xu Date: Fri, 1 Mar 2019 18:36:24 +0000 Subject: [PATCH] [PGO] Remove the default parameter in ProfileSummary API. NFC Remove the default parameter in setProfileSummary() and getSummary(). This is a follow-up of r355131. llvm-svn: 355216 --- include/llvm/IR/Module.h | 4 +--- include/llvm/ProfileData/InstrProfReader.h | 3 +-- 2 files changed, 2 insertions(+), 5 deletions(-) diff --git a/include/llvm/IR/Module.h b/include/llvm/IR/Module.h index 40c0db5e00c..f458680cfe1 100644 --- a/include/llvm/IR/Module.h +++ b/include/llvm/IR/Module.h @@ -869,9 +869,7 @@ public: /// @{ /// Attach profile summary metadata to this module. - // TODO: Remove the default paramter. - void setProfileSummary(Metadata *M, - ProfileSummary::Kind Kind = ProfileSummary::PSK_Instr); + void setProfileSummary(Metadata *M, ProfileSummary::Kind Kind); /// Returns profile summary metadata. When IsCS is true, use the context /// sensitive profile summary. diff --git a/include/llvm/ProfileData/InstrProfReader.h b/include/llvm/ProfileData/InstrProfReader.h index 25f9df03321..0aaa7cfff4f 100644 --- a/include/llvm/ProfileData/InstrProfReader.h +++ b/include/llvm/ProfileData/InstrProfReader.h @@ -501,8 +501,7 @@ public: /// Return the profile summary. /// \c UseCS indicates whether to use the context-sensitive summary. - // TODO: removed the defualt parameter. - ProfileSummary &getSummary(bool UseCS = false) { + ProfileSummary &getSummary(bool UseCS) { if (UseCS) { assert(CS_Summary && "No context sensitive summary"); return *(CS_Summary.get());